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

Implement switch in front and back camera in JS SDK for mobile platforms #2022

Closed
mekya opened this issue Apr 8, 2020 · 7 comments
Closed
Assignees

Comments

@mekya
Copy link
Contributor

mekya commented Apr 8, 2020

  • Provide a method in JS SDK to switch cameras in the system(back/forth or other cameras)
  • Switching cameras should be working in non-broadcasting and broadcasting state.
    • In broadcasting state, streaming should not be stopped while switching cameras
  • Create a radio button in WebRTC Publish demo that user can choose cameras
  • Please check turnon/turnoff camera works for all cameras
@RodrigoRVieira
Copy link

Facing exactly the same issue here with latest SDK/server. Fix please.

@Glfarley
Copy link

Inconvenient to have to disable/enable cameras in Windows to switch them. Cannot choose any cameras at all in iOS 13.4.1!

@mekya
Copy link
Contributor Author

mekya commented May 18, 2020

This feature is implemented and it's in PR. Unfortunately, travis has some problem for a couple of days that forces java 11 installation. This is why we don't merge it into the master.

@Glfarley
Copy link

Thanks. Will this also fix the problem where AMS does not choose any camera at all on my iPhone 11?

@mekya mekya assigned mekya and unassigned Monata May 18, 2020
@mekya mekya closed this as completed Jun 19, 2020
@0x62
Copy link

0x62 commented Jun 3, 2021

This feature appears to be broken on Firefox - both in my implementation and on the demo application.

Enterprise Edition 2.3.2 20210422_0456
Firefox Developer 89.0b15

This can be fixed by using exact object on lines 13 & 16 of switchVideoCameraCapture:

this.mediaConstraints.video.deviceId = { exact: deviceId };
...
deviceId: { exact: deviceId },

Screenshot 2021-06-03 at 13 49 45

@mekya
Copy link
Contributor Author

mekya commented Jun 6, 2021

Thank you so much for your update @0x62.

I'm opening this issue to fix

@mekya
Copy link
Contributor Author

mekya commented Jun 8, 2021

Thank you @0x62 your contribution, you're great. We've sent the PR for that

@mekya mekya closed this as completed Sep 13, 2021
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

5 participants