Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ff130 csp report to #24047

Merged
merged 3 commits into from
Aug 12, 2024
Merged

Ff130 csp report to #24047

merged 3 commits into from
Aug 12, 2024

Conversation

hamishwillee
Copy link
Collaborator

@hamishwillee hamishwillee commented Aug 6, 2024

FF130 supports the CSP report-to directive in https://bugzilla.mozilla.org/show_bug.cgi?id=1391243 behind a preference.

in https://bugzilla.mozilla.org/show_bug.cgi?id=1860588:

There is also some Reporting API stuff that is quite related to document. I'll do that separately.

I'm confirming this in https://bugzilla.mozilla.org/show_bug.cgi?id=1391243#c11 and some related stuff, but what I have here should be correct based on the IDL and info in bugs.

Related docs work can be tracked in mdn/content#35279

@github-actions github-actions bot added the data:http 🚠 Compat data for HTTP features. https://developer.mozilla.org/docs/Web/HTTP label Aug 6, 2024
@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Aug 12, 2024
@@ -10,7 +10,14 @@
"chrome_android": "mirror",
"edge": "mirror",
"firefox": {
"version_added": false
"version_added": "130",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"chrome_android": "mirror",
"edge": "mirror",
"firefox": {
"version_added": "130",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added in https://bugzilla.mozilla.org/show_bug.cgi?id=1860588

Note, isn't clear if headers are behind prefs or not (i.e. no clear signal about this as there is in IDL), but they aren't useful if the associated code gated by this isn't preference, so it makes sense to add the pref.

"spec_url": "https://w3c.github.io/reporting/#header-field-registration",
"support": {
"chrome": {
"version_added": "96"
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -32,8 +39,8 @@
},
"status": {
"experimental": false,
"standard_track": true,
"deprecated": false
"standard_track": false,
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has been removed from the spec, superseded by Reporting-Endpoints

@@ -780,7 +780,14 @@
"chrome_android": "mirror",
"edge": "mirror",
"firefox": {
"version_added": false
"version_added": "130",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -11,7 +11,14 @@
"chrome_android": "mirror",
"edge": "mirror",
"firefox": {
"version_added": false
"version_added": "130",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hamishwillee hamishwillee marked this pull request as ready for review August 12, 2024 02:27
@hamishwillee hamishwillee mentioned this pull request Aug 12, 2024
5 tasks
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Hamish! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:http 🚠 Compat data for HTTP features. https://developer.mozilla.org/docs/Web/HTTP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants