Closed Bug 1336654 Opened 3 years ago Closed 3 years ago

Permafailing dom/security/test/contentverifier/browser_verify_content_about_newtab.js | Valid remote newtab page must have built-in CSP. - Got {}, expected {"csp-policies":[{"report-only":false,"script-src":["https://example.com","'unsafe-inline'"]

Categories

(Core :: DOM: Security, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox-esr45 --- unaffected
firefox51 --- fixed
firefox52 --- fixed
firefox-esr52 --- fixed
firefox53 --- fixed
firefox54 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: franziskus)

References

Details

(Whiteboard: [stockwell fixed])

Attachments

(1 file)

Filed by: wkocher [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=74451524&repo=autoland

https://queue.taskcluster.net/v1/task/IZ4hTl6GQfuCiszvI-s7wQ/runs/0/artifacts/public/logs/live_backing.log

This just started failing out of the blue today on both inbound and autoland. Retriggers of previously-passing runs are all failing.
Pushed by kwierso@gmail.com:
https://hg.mozilla.org/mozilla-central/rev/7aa5e444af0f
Skip the suddenly permafailing tests to reopen the tree a=me CLOSED TREE
Keeler, you've recently(ish) reviewed changes to this test. Any ideas?
Flags: needinfo?(dkeeler)
goodChain.pem expired yesterday:

        Validity
            Not Before: Nov 27 00:00:00 2014 GMT
            Not After : Feb  4 00:00:00 2017 GMT

https://dxr.mozilla.org/mozilla-central/rev/1d025ac534a6333a8170a59a95a8a3673d4028ee/dom/security/test/contentverifier/goodChain.pem

For this reason, all tress with this test should be affected.
Blocks: 1263793
Flags: needinfo?(dkeeler) → needinfo?(franziskuskiefer)
This is related to bug 1332131 that replaced expiring certificates. But this one needs some manual work. (Sorry for this patch here but MozReview isn't working for me atm...)
Assignee: nobody → franziskuskiefer
Flags: needinfo?(franziskuskiefer)
Attachment #8833908 - Flags: review?(mgoodwin)
Comment on attachment 8833908 [details] [diff] [review]
fix-cs-tests.patch

LGTM
Attachment #8833908 - Flags: review?(mgoodwin) → review+
Pushed by franziskuskiefer@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/507fcddedb1f
update expired certs and signatures for content signature tests, r=mgoodwin
https://hg.mozilla.org/mozilla-central/rev/507fcddedb1f
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Whiteboard: [checkin-needed-aurora][checkin-needed-beta][checkin-needed-release]
https://hg.mozilla.org/releases/mozilla-release/rev/34b53a3ed396
Flags: in-testsuite+
Whiteboard: [checkin-needed-aurora][checkin-needed-beta][checkin-needed-release] → [checkin-needed-aurora][checkin-needed-beta]
https://hg.mozilla.org/releases/mozilla-beta/rev/cbe7e2f489d3
Whiteboard: [checkin-needed-aurora][checkin-needed-beta] → [checkin-needed-aurora]
Whiteboard: [stockwell fixed]
You need to log in before you can comment on or make changes to this bug.