-
Notifications
You must be signed in to change notification settings - Fork 62
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
enumerateDevices should only provide device info permission for granted device types #645
Comments
I often work with several cameras and audio devices simultaneously. In these scenarios, I don't think a per-device info grant makes sense, as that prohibits the application from showing a list of devices to the user to choose from. We already have the problem of prompting the user what initial device they'd like to use. While for many video conferencing style applications, leaving the default device is appropriate, more advanced use cases really do require this device information exposed to the application. It's my understanding that the upcoming Permissions API will help with this, as we will be able to request what's needed up-front? In either case, I think the suggestion of limiting to device type would be fine. |
Issue specific to enumerateDevices. Might want to consider devicechange as a separate issue. |
The spec allows to expose devices through enumerateDevices after a successful getUserMedia call.
It makes sense to restrict it to either device or device type.
Per-device would be good to have but might depend on the actual browser implementation.
Per device type seems like the minimum bar.
In particular, if a user agent is asking "Allow access to camera", it does not make real sense to leak microphones.
The text was updated successfully, but these errors were encountered: