-
Notifications
You must be signed in to change notification settings - Fork 617
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
Video.js plugin does not use the provided iceServers #6292
Comments
Hi @wheelbarrow777, Thank you for the report. We've put it to the backlog. If you want us to make this a priority, please reach out to [email protected] or continue your discussion with the support channel. Cheers |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Short description
In https://github.com/ant-media/videojs-webrtc-plugin, the iceServers provided in the
player.src()
call is not used in the NPM package.In https://www.npmjs.com/package/@antmedia/videojs-webrtc-plugin?activeTab=code, navigate to
/dist/videojs-webrtc-plugin.js
. At line 3858, theRTCPeerConnection
config used isthis.peerconnection_config
.this.peerconnection_config
is defined once and never changed at line 2869.Correct behavior would be to change line 3858 to
this.remotePeerConnection[streamId] = new RTCPeerConnection(this.pcConfig);
, asthis.pcConfig
is the configuration parsed from the src.The text was updated successfully, but these errors were encountered: