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 features for permissions policy and feature policy #2661

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ddbeck
Copy link
Collaborator

@ddbeck ddbeck commented Feb 17, 2025

This PR adds a feature for permissions policy and a discouraged feature for feature policy. This is a big mess, so these are draft features.

Unfortunately, BCD doesn't accurately record the data for these features and it it's a bit of work to unwind this. I propose that we add these as drafts for now and, at the time we merge this, file an issue on BCD that requests Feature-Policy and Permissions-Policy be separated into separate keys and see where things go from there.

The full backstory:

Given this story, we cannot accurately calculate either feature's implementation status in a way that accords with caniuse or, in my opinion, confusing developers (because let me tell you, I was confused about this and I knew most of the backstory already).

@github-actions github-actions bot added the tools and infrastructure Project internal tooling, such as linters, GitHub Actions, or repo settings label Feb 17, 2025
@ddbeck
Copy link
Collaborator Author

ddbeck commented Feb 17, 2025

cc: @Elchi3 you might have opinions about this one, since I expect it to impact BCD as well.

spec: https://w3c.github.io/webappsec-permissions-policy/
caniuse: permissions-policy
compat_features:
- http.headers.Permissions-Policy
Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah. I asked about this at the time and it seems like at some point somebody thought about it but nothing else came of it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tools and infrastructure Project internal tooling, such as linters, GitHub Actions, or repo settings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants