Closed
Bug 1140739
Opened 10 years ago
Closed 10 years ago
browser_net_security-details.js failure from the NSS 3.18 RC0 landing
Categories
(DevTools :: Netmonitor, defect)
DevTools
Netmonitor
Tracking
(firefox38 fixed, firefox39 fixed)
RESOLVED
FIXED
Firefox 39
People
(Reporter: philor, Assigned: sjakthol)
References
Details
Attachments
(1 file)
(deleted),
patch
|
vporof
:
review+
|
Details | Diff | Splinter Review |
The NSS update from bug 1137470 increased NSS's default maximum supported version of TLS from 1.0 to 1.2, which caused http://mxr.mozilla.org/mozilla-central/source/browser/devtools/netmonitor/test/browser_net_security-details.js#40 to fail because it expects exactly a TLSv1 connection without having done anything to ensure it gets one.
I'll land a change to increase that to expecting exactly a TLSv1.2 connection once I have the number from this bug to use in a commit message, but the test really needs to do something else, either ensure that it will get a particular version or not test that it has gotten a particular version.
Reporter | ||
Comment 1•10 years ago
|
||
Keywords: leave-open
Comment 2•10 years ago
|
||
We'll also need this on Aurora, because 3.18 RTM will have to land there.
status-firefox38:
--- → affected
Reporter | ||
Comment 3•10 years ago
|
||
You can land test-only changes on aurora with a=test-only, no need to worry about that.
Assignee | ||
Comment 4•10 years ago
|
||
Here's a patch that makes the test work with different TLS protocol versions and cipher suites. It changes the test not to assume any specific values but to check if the values look valid i.e. the protocol version starts with "TLS" and the cipher suite starts with "TLS_".
This should be quite a bit more future proof.
Try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7eb480adbb12
Reporter | ||
Comment 5•10 years ago
|
||
Comment 6•10 years ago
|
||
Comment on attachment 8574317 [details] [diff] [review]
netmonitor-security-test-fix.patch
Review of attachment 8574317 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM
Attachment #8574317 -
Flags: review?(vporof) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 7•10 years ago
|
||
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Assignee | ||
Updated•10 years ago
|
Keywords: leave-open
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 39
Comment 9•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/1adf44fcb513
(This is the combination of Phil's first patch, and the later patch on top.)
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•