Add idl to set up public key pins to PublicKeyPinningService

RESOLVED DUPLICATE of bug 787133

Status

()

RESOLVED DUPLICATE of bug 787133
4 years ago
4 years ago

People

(Reporter: cviecco, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
This bug is to add an idl to and a memory only store PublicKeyPinningService so that public key pins can be added via xpcom calls. The store must be thread safe.
(Reporter)

Updated

4 years ago
Blocks: 787133
(Reporter)

Comment 1

4 years ago
Created attachment 8466465 [details] [diff] [review]
add-pkservice-idl
(Reporter)

Comment 2

4 years ago
Comment on attachment 8466465 [details] [diff] [review]
add-pkservice-idl

Review of attachment 8466465 [details] [diff] [review]:
-----------------------------------------------------------------

So I was thinking about this interface. What do you think (maybe the insert is redundant?)
Attachment #8466465 - Flags: feedback?(dkeeler)
Comment on attachment 8466465 [details] [diff] [review]
add-pkservice-idl

Review of attachment 8466465 [details] [diff] [review]:
-----------------------------------------------------------------

Why not just use nsISiteSecurityService? It already can parse headers similar to HPKP.
Attachment #8466465 - Flags: feedback?(dkeeler) → feedback-
(Reporter)

Comment 4

4 years ago
(In reply to David Keeler (:keeler) [use needinfo?] from comment #3)
> Comment on attachment 8466465 [details] [diff] [review]
> add-pkservice-idl
> 
> Review of attachment 8466465 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Why not just use nsISiteSecurityService? It already can parse headers
> similar to HPKP.

Because this inferface is specific to public pins and I am no interested in parsing at this
moment. Also, nsISiteSecurityService is not thread safe (to my understanding) for accesing its
information as it currently uses the site preferences.
(Reporter)

Updated

4 years ago
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 787133
You need to log in before you can comment on or make changes to this bug.