Closed
Bug 1173679
Opened 10 years ago
Closed 10 years ago
Add tests for the "security.OCSP.enabled" pref
Categories
(Core :: Security: PSM, defect)
Core
Security: PSM
Tracking
()
RESOLVED
FIXED
mozilla47
| Tracking | Status | |
|---|---|---|
| firefox47 | --- | fixed |
People
(Reporter: Cykesiopka, Assigned: Cykesiopka)
References
Details
Attachments
(1 file)
Bug 1010068 added a new "ocspEVOnly" (i.e. 2) option for the "security.OCSP.enabled" pref, but didn't add any tests.
AFAICT the previous options weren't tested either, but tests should be added to make sure the new functionality added in Bug 1010068 works, and to make sure both the new and old options keep working.
|
Assignee | |
Updated•10 years ago
|
Assignee: nobody → cykesiopka.bmo
Status: NEW → ASSIGNED
OS: Unspecified → All
Hardware: Unspecified → All
|
|
Assignee | |
Comment 1•10 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/37099/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/37099/
Attachment #8724614 -
Flags: review?(dkeeler)
|
|
||
Comment 2•10 years ago
|
||
Comment on attachment 8724614 [details]
MozReview Request: Bug 1173679 - Add tests for the "security.OCSP.enabled" pref.
https://reviewboard.mozilla.org/r/37099/#review33767
Solid work - thanks for taking care of this.
Attachment #8724614 -
Flags: review?(dkeeler) → review+
|
|
Assignee | |
Comment 3•10 years ago
|
||
No problem. Thanks for the review!
https://treeherder.mozilla.org/#/jobs?repo=try&revision=6c4a9d33172b
Keywords: checkin-needed
Keywords: checkin-needed
|
||
Comment 5•10 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in
before you can comment on or make changes to this bug.
Description
•