Open Bug 1273241 Opened 5 years ago Updated 3 years ago

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


(Testing :: web-platform-tests, defect)

48 Branch
Not set


(Not tracked)


(Reporter: KWierso, Unassigned)


(Keywords: leave-open, 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)
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:

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:


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.

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.