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

Allow for the query algorithm to return prompt or denied when document is not allowed to use #458

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aselya
Copy link

@aselya aselya commented Jan 27, 2025

Updated language to allow the query algorithm to return prompt ordenied helps protect the user from exposing their available features and helps prevent retaliation against the user from developers.

closes #388

The following tasks have been completed:

  • Modified Web platform tests (link)

Implementation commitment:

  • WebKit (link to issue)
  • Blink (link to issue)
  • Gecko (link to issue)

Preview | Diff

…ad of "denied"

Allowing the query algorithm to return `prompt` or`denied` helps protect the user from exposing their available features and helps prevent retaliation against the user from developers.
@miketaylr miketaylr self-requested a review January 28, 2025 15:27
@miketaylr
Copy link
Member

There is some prior art at https://privacycg.github.io/storage-access/#permissions-integration and https://privacycg.github.io/requestStorageAccessFor/#permissions-integration

Conceptually, WDYT @marcoscaceres, should we pull this into permissions, or just add a note saying powerful features can do this in their own permission query algorithms?

@johannhof
Copy link
Member

@aselya can you elaborate a bit more on why you think exposing Permissions Policy state (which is "allowed to use") would lead to retaliation against the user?

I could see an argument for why this technically exposes cross-origin information, but that seems by design, the same way that, say, the sandbox argument is observable by a cross-origin iframe. Also, that doesn't seem like something that should be implementation-defined. :)

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

Successfully merging this pull request may close these issues.

Allow returning "prompt" rather than "denied"
3 participants