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

WebRTC adaptor does not use the turn server passed to it's constructor from embedded web player #6370

Closed
Mohit-3196 opened this issue May 28, 2024 · 4 comments

Comments

@Mohit-3196
Copy link
Contributor

Mohit-3196 commented May 28, 2024

Short description

When we pass a TURN server to the embedded-player.js it does not use the turn server passed to its constructor and uses the default STUN server.

Environment

  • Java version: 17
  • Ant Media Server version: 2.9.0

Steps to reproduce

  1. Pass the TURN server to the embedded-player.js
  2. Play the stream using the play.html
  3. Check the webrtc internals

Expected behavior

The webrtc adaptor should use the ice server passed to its constructor.

Actual behavior

The webrtc adaptor does not use the ice server passed to its constructor but continues to use the default STUN server.

Logs

Place logs on pastebin or elsewhere and put links here

Ask your questions on Ant Media Github Discussions

@lastpeony
Copy link
Contributor

@lastpeony lastpeony changed the title WebRTC adaptor does not use the turn server passed to it's constructor WebRTC adaptor does not use the turn server passed to it's constructor from embedded player May 28, 2024
@lastpeony lastpeony changed the title WebRTC adaptor does not use the turn server passed to it's constructor from embedded player WebRTC adaptor does not use the turn server passed to it's constructor from embedded web player May 28, 2024
@mekya mekya moved this to Next Sprint in Ant Media Server May 31, 2024
@mekya mekya moved this from Next Sprint to 🏗 In progress in Ant Media Server Jun 5, 2024
@mekya
Copy link
Contributor

mekya commented Jun 21, 2024

Hi @lastpeony,

Thank you for the fix.

To make this fix available in the Ant Media Server, we need the following things as well.

  1. Update the Web-Player for the version in videojs-webrtc-plugin.
  2. Update the StreamApp for the new version of Web-Player

At this time, I'm going ahead and creating the versions and PRs to let you see it. If you already know, then accept this as a reminder.
Next time, I expect you to make it ;)

Thank you again for your contributions.

Let me know if you have any questions or want to add some things

Cheers
Oguz

@mekya
Copy link
Contributor

mekya commented Jun 21, 2024

Just want to add one thing, I've also added test codes for the fix.
Check it out
https://github.com/ant-media/videojs-webrtc-plugin/pull/25/files#diff-dbd99cd221621a3bab7c898fb51a53d2bfbf88eb6bb495ccaea81222837cf18d

@mekya
Copy link
Contributor

mekya commented Jun 21, 2024

This issue can be closeable in the sprint meeting @burak-58

@mekya mekya closed this as completed Jun 24, 2024
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Ant Media Server Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants