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

Vibration should require user's permission #10

Open
andrey-logvinov opened this issue Jun 15, 2016 · 6 comments
Open

Vibration should require user's permission #10

andrey-logvinov opened this issue Jun 15, 2016 · 6 comments
Labels

Comments

@andrey-logvinov
Copy link

It looks like an omission that Notifications API requires user's permission to issue notifications which also include vibration patterns, but the Vibration API doesn't require any sort of permission. There is a provision in the "Security and privacy considerations" section that UA should provide mechanism for disabling the API but there is already a standard mechanism for that purpose, the Permissions API. I suggest adding explicit user permission requirement instead of the general provision of an unspecified disabling mechanism.

@anssiko
Copy link
Member

anssiko commented Jun 15, 2016

The Notifications API permission model does not special case vibration. The same "granted" permission is required for all notifications regardless of the use of vibration.

@andrey-logvinov
Copy link
Author

andrey-logvinov commented Jun 15, 2016

This is correct, but to vibrate the device through notification, notification permission is still required. On the other hand, to vibrate the device via Vibration API, no permission is required at all. This does cause some concern, e.g. https://bugs.chromium.org/p/chromium/issues/detail?id=507703

The specification already contains provision that UA should inform the user when the Vibration API is being used and allow to disable vibration on per-origin basis or globally, but does not specify the mechanism for informing the user and disabling the vibration. I suggest considering explicitly specifying a permission for use with Vibration API, because permission requests both inform the user that a certain feature is going to be used and allow to disable the use of the feature on per-origin basis.

@anssiko
Copy link
Member

anssiko commented Jun 16, 2016

Agreed to defer to v2 on today's call.

@anssiko anssiko added the v2 label Jun 16, 2016
@lknik
Copy link

lknik commented Jun 16, 2016

We also could actually give a MUST in the privacy considerations (i.e. ask for permissions). Or just stipulate this is subject to permissions. For v2.

@andrey-logvinov
Copy link
Author

A MUST will invalidate at least the current Chromium implementation which neither informs the user nor provides a mechanism to disable vibration. So this is no really different from adding a permission in regard of existing implementations validity, which is the reason this issue is deferred to v2.

@anssiko
Copy link
Member

anssiko commented Jun 17, 2016

Correct. The consensus was to defer this feature to v2, and publish a Proposed Edited Recommendation (PER) that reflects the current implementations (aka reality), after integrating the editorial change #7.

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

No branches or pull requests

3 participants