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

Add Permissions-Policy #18250

Merged
merged 3 commits into from Nov 21, 2022
Merged

Conversation

chrisdavidmills
Copy link
Collaborator

@chrisdavidmills chrisdavidmills commented Nov 20, 2022

Summary

The Feature-Policy header changed to Permissions-Policy some time ago. This PR adds Permission-Policy BCD, keeping the existing Feature-Policy data but classifying it as support under an alternative name.

This is part of the work described at mdn/content#22347

See my research document at https://docs.google.com/document/d/14xHANU3n43PfJR_dCpEwVCiVYv12BAfGqOICNvd79R4/edit#heading=h.mahbkxhoku5t for a list of the Permissions-Policy directives I am intending to implement on MDN as part of this work. This gives us parity with what was previously documented under Feature-Policy, but it adds a few more that are known to be supported, and it removes mention of some that were obviously being worked on at some point, but never seem to have gotten any traction.

Note — I have not yet deleted the Feature-Policy.json file, so that the existing docs don't break if the new BCD is merged and imported sooner than the MDN content is updated. We can always delete that later on.

Test results and supporting details

New information was largely gotten from https://github.com/w3c/webappsec-permissions-policy/blob/main/features.md.

Related issues

@github-actions github-actions bot added the data:http 🚠 Compat data for HTTP features. https://developer.mozilla.org/docs/Web/HTTP label Nov 20, 2022
@chrisdavidmills chrisdavidmills changed the title Add permissions policy Add Permissions-Policy Nov 20, 2022
Copy link
Collaborator

@queengooborg queengooborg left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@queengooborg queengooborg merged commit fabdbdc into mdn:main Nov 21, 2022
@chrisdavidmills chrisdavidmills deleted the add-Permissions-Policy branch November 22, 2022 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

None yet

2 participants