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

Feature Policy: Disable all? #193

Closed
Bramzor opened this issue May 28, 2019 · 6 comments
Closed

Feature Policy: Disable all? #193

Bramzor opened this issue May 28, 2019 · 6 comments

Comments

@Bramzor
Copy link

Bramzor commented May 28, 2019

Documentation describes how to disable or configure specific features but I was wondering why wouldnt we just disable all and enable only the items we need? Or am I missing something?

@EvanHahn
Copy link
Member

To quote MDN:

If you do not specify a policy for a feature, then a default allowlist will be used. The default allowlist is specific to each feature.

A "disable everything" feature is an interesting idea...I assume that's something you'd be interested in?

@Bramzor
Copy link
Author

Bramzor commented May 29, 2019

Do we know the content of this default allowlist?
Yes, I would prefer to actually disable everything as on a normal website or service you do not need anything like camera, micro, geolocation etc. And per security best practices, it's always better to deny as much as possible. Although I tried disabling vibrate yesterday and apparently Chrome (desktop) was unaware of this feature so it started showing warnings. This might be a reason to not disable everything.

@EvanHahn
Copy link
Member

I'm not sure, but another quote from MDN:

For each policy-controlled feature, the browser maintains a list of origins for which the feature is enabled, known as an allowlist.

The spec mentions default allowlists, but it doesn't give specifics.

Unfortunately, I don't know of a great way to disable everything without giving those warnings. If there are any features that the browser doesn't recognize, it will give a warning (I assume, I haven't tested every browser).

@Bramzor
Copy link
Author

Bramzor commented May 29, 2019

There is a post talking about a possible default feature: w3c/webappsec-permissions-policy#189 . Apparently the allowlist is configured and configurable in the browser.

@EvanHahn
Copy link
Member

EvanHahn commented Jun 4, 2019 via email

@EvanHahn
Copy link
Member

Let's move this discussion to helmetjs/feature-policy#6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants