Closed
Bug 1257969
Opened 8 years ago
Closed 8 years ago
update test_pinning_dynamic.js test certificates to not use subject common name for name information
Categories
(Core :: Security: PSM, defect)
Core
Security: PSM
Tracking
()
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: keeler, Assigned: keeler)
Details
Attachments
(1 file)
The test_pinning_dynamic.js test certificates have the form 'subject:<something>.example.com' and no other naming information. Depending on the default value of the pref that will be set in bug 1245280, this will not work in the future. These certificates should just use subjectAlternativeName extensions.
Assignee | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/41153/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/41153/
Attachment #8732383 -
Flags: review?(jjones)
Updated•8 years ago
|
Attachment #8732383 -
Flags: review?(jjones) → review+
Comment 2•8 years ago
|
||
Comment on attachment 8732383 [details] MozReview Request: bug 1257969 - update test_pinning_dynamic.js test certificates to not use subject common name for name information r?jcj https://reviewboard.mozilla.org/r/41153/#review37739 ![LGTM](http://lgtm.in/p/MwCQCbseq) Looks good to me.
Assignee | ||
Comment 3•8 years ago
|
||
Thanks! https://treeherder.mozilla.org/#/jobs?repo=try&revision=dcdd51be4388
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/81d4f44d145e
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•