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

push and scan should be gated by user activation #425

Closed
beaufortfrancois opened this issue Oct 30, 2019 · 5 comments · Fixed by #430
Closed

push and scan should be gated by user activation #425

beaufortfrancois opened this issue Oct 30, 2019 · 5 comments · Fixed by #430

Comments

@beaufortfrancois
Copy link
Collaborator

It looks like we're going with permission prompt. Therefore we should update spec to make sure NDEFReader.scan and NDEFWriter.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

@zolkis
Copy link
Contributor

zolkis commented Oct 30, 2019

So this is a more specific formulation of #368?
Since the HTML user activation PR has not landed yet, there is uncertainty what prose to use.
Should we encapsulate this anyway under "obtaining permission"?

BTW the whole security section needs some love, see #420

@beaufortfrancois
Copy link
Collaborator Author

I think @mustaqahmed would know.

@mustaqahmed
Copy link
Member

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?

@jyasskin
Copy link
Member

w3c/permissions#107 has resistance from Firefox folks to the idea of requiring user activation. w3c/permissions#194 is also related.

@beaufortfrancois
Copy link
Collaborator Author

@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.

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 a pull request may close this issue.

4 participants