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

Rely upon FedCM as a trust signal for storage access, rather than modifying requestStorageAccess #8

Closed
bvandersloot-mozilla opened this issue Mar 20, 2024 · 3 comments

Comments

@bvandersloot-mozilla
Copy link
Collaborator

No description provided.

@bvandersloot-mozilla
Copy link
Collaborator Author

Actually, I don't think the connected accounts set makes sense as a trust signal for this Credential, since it can be stored and collected. This permits finer control over where storage access can be granted.

The key thing I think should be preserved here is whether or not the domain is preventing silent access. If a site goes out of its way to call navigator.credentials.preventSilentAccess(), then we shouldn't be using things from the credential store without a dialog.

We could add that as a constraint to the proposal from Google: https://github.com/explainers-by-googlers/storage-access-for-fedcm

or we could match that explainer to this syntax.

@johannhof - this was really your issue - thoughts?

@johannhof
Copy link
Member

That makes sense to me, I think. I filed the issue above to track this.

@bvandersloot-mozilla bvandersloot-mozilla added the agenda+ Request to add this issue to the agenda of our next telcon or F2F label May 8, 2024
@bvandersloot-mozilla
Copy link
Collaborator Author

I think this is resolved, but leaving open until we discuss in the CG.

@bvandersloot-mozilla bvandersloot-mozilla removed the agenda+ Request to add this issue to the agenda of our next telcon or F2F label May 15, 2024
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

No branches or pull requests

2 participants