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

Firefox Support #6

Closed
adwpc opened this issue May 5, 2020 · 6 comments
Closed

Firefox Support #6

adwpc opened this issue May 5, 2020 · 6 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@adwpc
Copy link
Contributor

adwpc commented May 5, 2020

Firefox Support

This is the Browser Market Share Worldwide:
https://gs.statcounter.com/browser-market-share#monthly-201903-202003-bar
Chrome has > 60% share worldwide

Ion support chrome well, but on firefox, we need to do many work(low priority):

  1. use _payloadModify(in Client.js) to modify the sdp when publish or subscribe

on chrome, we use 111 for opus, 96 for vp8 and 102 for h264 ,as a reference
we should do this for firefox

  1. test audio and video between firefox to firefox , firefox to chrome

  2. other js bug on firefox

If anyone can help, that would be awsome!

@tarrencev tarrencev added enhancement New feature or request help wanted Extra attention is needed labels May 6, 2020
@Leeaandrob
Copy link

there any news abotu this issue?

@OrlandoCo
Copy link
Contributor

OrlandoCo commented May 12, 2020

Why it's not possible to set the webrtc transceiver direction to "sendonly" in Pion? I tried to set it but it sends an error that only "recvonly" and "sendrecv" is supported.

Change the client to "sendrecv" in the subscription (including the changes in pion) makes that all browsers doesn't display the video.

@tarrencev
Copy link
Contributor

@OrlandoCo I'm not sure. pion/webrtc might be a better place to ask.

@tarrencev
Copy link
Contributor

@Leeaandrob No updates yet. Feel free to contribute!

@tarrencev
Copy link
Contributor

@jbrady42 should we close this?

@jbrady42
Copy link
Contributor

This was implemented server side in the SFU.

dreamerns added a commit to dreamerns/ion-sdk-js that referenced this issue Oct 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants