Several CSP web platform tests are failing after the latest update from bug 1273176

NEW
Unassigned

Status

defect
3 years ago
6 months ago

People

(Reporter: KWierso, Unassigned)

Tracking

({leave-open})

48 Branch
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [test disabled] )

I'm going to be disabling them shortly.
I'm sure there's a more precise way to disable these on only the affected platforms, but they're currently holding the tree closed.
Flags: needinfo?(james)
Keywords: leave-open
This probably needs triage from the security team...
Flags: needinfo?(ckerschb)
(In reply to Boris Zbarsky [:bz] (still a bit busy) (if a patch has no decent message, automatic r-) from comment #4)
> This probably needs triage from the security team...

Thanks for letting me know. I'll have someone look at that.
Francois, Wennie mentioned you might be able to take a look at this one. If not, please let me know!
Flags: needinfo?(ckerschb) → needinfo?(francois)
Here's what I found so far.

Tests that fail:

https://github.com/w3c/web-platform-tests/blob/master/content-security-policy/blink-contrib/self-doesnt-match-blob.sub.html
https://github.com/w3c/web-platform-tests/blob/master/content-security-policy/blink-contrib/star-doesnt-match-blob.sub.html
https://github.com/w3c/web-platform-tests/blob/master/content-security-policy/blink-contrib/worker-connect-src-allowed.sub.html
https://github.com/w3c/web-platform-tests/blob/master/content-security-policy/blink-contrib/worker-script-src.sub.html

I suspect they fail because we lack SecurityPolicyViolationEvent (bug 1302962). I've got too much on this week and next, but I'll try to confirm after that.

Tests that are no longer in the upstream repo:

testing/web-platform/meta/content-security-policy/frame-ancestors/multiple-frames-meta-ignored.sub.html.ini
testing/web-platform/meta/content-security-policy/frame-ancestors/multiple-frames-self-allowed.sub.html.ini
testing/web-platform/meta/content-security-policy/frame-ancestors/single-frame-self-allowed.sub.html.ini

They have been replaced with new ones, so it's probably not worth looking into these.
Flags: needinfo?(francois)
Flags: needinfo?(james)
Also with SecurityPolicyViolationEvent enabled, these blob tests fail because we don't 'rename' directives.
See: https://w3c.github.io/webappsec-csp/#effective-directive-for-a-request

SecurityPolicyViolation event should report 'worker-src' as violation-directive, but we report 'child-src' instead.
Do we have a bug filed for the naming of CSP directives?
Flags: needinfo?(ckerschb)
(In reply to Andrea Marchesini [:baku] from comment #9)
> Do we have a bug filed for the naming of CSP directives?

I suppose that should be Bug 1192684 - Content Security Policy reports should include "effective-directive" and "status-code".
Flags: needinfo?(ckerschb)

The leave-open keyword is there and there is no activity for 6 months.
:jgraham, maybe it's time to close this bug?

Flags: needinfo?(james)
Flags: needinfo?(james)
Whiteboard: [test disabled]
You need to log in before you can comment on or make changes to this bug.