-
Notifications
You must be signed in to change notification settings - Fork 67
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
push and scan should be gated by user activation #425
Comments
So this is a more specific formulation of #368? BTW the whole security section needs some love, see #420 |
I think @mustaqahmed would know. |
Yes, "should be gated by user activation" is the right solution, addresses the concerns I had with #368. Re fixing this spec issue: if the plan is to fix this before the user activation PR lands (hopefully in a few weeks), please use "triggered by user activation" for now and cc me in a new issue to rephrase it. Or you can wait. IFAIK Chrome's permission dialog is gated by user activation, at least for the cases I checked. So the chain "NFC gated by permissions gated by user activation" makes perfect sense to me. I am assuming there is no "piece" of the NFC API that does not need a permission yet should be gated by user activation. As a side note, I see that the Permissions API does not specify the dependence on user activation. @jyasskin, does it have an issue to cover this? |
w3c/permissions#107 has resistance from Firefox folks to the idea of requiring user activation. w3c/permissions#194 is also related. |
@mustaqahmed We went for "triggered by user activation" for now. I'll open a new issue that I'll assign to you to rephrase it when user activation PR lands. |
It looks like we're going with permission prompt. Therefore we should update spec to make sure
NDEFReader.scan
andNDEFWriter.push
are gated by user activation.@zolkis I think gh-pages...beaufortfrancois:user-activation is not enough. There's already some permission and user prompts section at https://w3c.github.io/web-nfc/#permissions-and-user-prompts which may also need some love.
Could you take a look?
This should solve #368
The text was updated successfully, but these errors were encountered: