Video.js plugin does not use the provided iceServers

[wheelbarrow777]

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.

player.src({
  src: 'ws://localhost:5080/LiveApp/stream1.webrtc',
  iceServers: '[ { "urls": "stun:stun1.l.google.com:19302" } ]'
});

In https://www.npmjs.com/package/@antmedia/videojs-webrtc-plugin?activeTab=code, navigate to /dist/videojs-webrtc-plugin.js. At line 3858, the RTCPeerConnection config used is this.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);, as this.pcConfig is the configuration parsed from the src.

[mekya]

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 contact@antmedia.io or continue your discussion with the support channel.

Cheers
Oguz