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

Rename Feature-Policy header to Permissions-Policy #51

Closed
yosifeldman opened this issue Jul 21, 2020 · 8 comments
Closed

Rename Feature-Policy header to Permissions-Policy #51

yosifeldman opened this issue Jul 21, 2020 · 8 comments
Labels
pending wait for browser support

Comments

@yosifeldman
Copy link

According to the following link, the Feature-Policy header is now renamed to the Permissions-Policy.

https://www.w3.org/TR/permissions-policy-1/

https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Feature-Policy

Since yesterday (July 20 2020) started to get errors in Console in the Chrome:
"Error with Feature-Policy header: Unrecognized feature: 'display-capture'."

@bepsvpt bepsvpt added the pending wait for browser support label Jul 27, 2020
@J2TEAM
Copy link

J2TEAM commented Jul 30, 2020

Hi @bepsvpt! Can you please update it soon? It shows a lot of errors on my website 😢

image

@yosifeldman
Copy link
Author

yosifeldman commented Aug 3, 2020 via email

@jamesparkes
Copy link

any updates on this one?

@bepsvpt
Copy link
Owner

bepsvpt commented Aug 9, 2020

6.2.0 adds use-permissions-policy-header config key to feature-policy, set it to true will use Permissions-Policy header name.

https://github.com/bepsvpt/secure-headers/blob/6.2.0/config/secure-headers.php#L148-L159

@jamesparkes
Copy link

worked like a charm. thanks 😄

@richardnbanks
Copy link

The header gets changed correctly but the format of the values changed too.

https://github.com/w3c/webappsec-permissions-policy/blob/master/permissions-policy-explainer.md#appendix-big-changes-since-this-was-called-feature-policy

A policy which would previously have been expressed as:

Feature-Policy: fullscreen 'self' https://example.com https://another.example.com; geolocation *; camera 'none'

would now look like:

Permissions-Policy: fullscreen=(self "https://example.com" "https://another.example.com"), geolocation=*, camera=()

@HDVinnie
Copy link

Came across this as well after using https://observatory.mozilla.org/ . Hope to see it implemented and released.

@bepsvpt
Copy link
Owner

bepsvpt commented Oct 14, 2020

7.0.0 was released, if there is any problem with permissions-policy, feel free to reply on this issue.

Note: Feature-Policy was removed.

@bepsvpt bepsvpt closed this as completed Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending wait for browser support
Projects
None yet
Development

No branches or pull requests

6 participants