CSP require-sri-for does not block when CSP is in <meta> tag

RESOLVED FIXED in Firefox 50

Status

()

defect
P1
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: freddyb, Assigned: ckerschb)

Tracking

(Blocks 2 bugs)

unspecified
mozilla50
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox50 fixed)

Details

(Whiteboard: [domsecurity-active])

Attachments

(2 attachments)

(Reporter)

Description

3 years ago
This needs investigation and I have yet to figure out why.
It doesn't seem to be related to the loadType, but I might be wrong :)
I suppose in ::OnStreamComplete() we have to check if the contentpolicyType is TYPE_SCRIPT_PRELOAD, and if that is the case then we have to query and consult the preloadCSP |principal->GetPreloadCsp()| instead of the actual CSP. Oh joy!
Whiteboard: [domsecurity-backlog]
(Assignee)

Updated

3 years ago
Blocks: csp-w3c-3
(Assignee)

Updated

3 years ago
Blocks: SRI
(Assignee)

Updated

3 years ago
Priority: -- → P1
(Assignee)

Updated

3 years ago
Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
Whiteboard: [domsecurity-backlog] → [domsecurity-active]
Attachment #8767150 - Flags: review?(francois)
Attachment #8767150 - Flags: review?(francois) → review+
Attachment #8767152 - Flags: review?(francois) → review+

Comment 4

3 years ago
Pushed by mozilla@christophkerschbaumer.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2373b4f2f321
CSP require-sri-for does not block when CSP is in meta tag r=francois
https://hg.mozilla.org/integration/mozilla-inbound/rev/fe2cd5c40e73
Test require-sri-for in meta tag r=francois

Comment 5

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/2373b4f2f321
https://hg.mozilla.org/mozilla-central/rev/fe2cd5c40e73
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.