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

RTP Input for Audiobridges (Enhancement) #1360

Closed
austinmarkus opened this issue Sep 11, 2018 · 4 comments · Fixed by #2464
Closed

RTP Input for Audiobridges (Enhancement) #1360

austinmarkus opened this issue Sep 11, 2018 · 4 comments · Fixed by #2464

Comments

@austinmarkus
Copy link

It would be really useful for me to have the ability to input RTP to an audiobridge in a similar manner to how there is RTP output via the forwarder. The basic request is to have the ability to send an Opus encoded RTP stream from a non-browser such that it could be heard the members of the bridge.

If there is already a way to accomplish this that I have overlooked I'm open to suggestion.

Thank you for your amazing project.

@lminiero
Copy link
Member

This is currently unsupported in the AudioBridge. I can add it to the TODO list, but I honestly don't know when I'd be able to work on that (busy times!). PRs always welcome, of course!

An easier way to accomplish the same thing with what's available today would be by taking advantage of the Streaming plugin: create an audio-only Streaming mountpoint, have your users create a PeerConnection with that (e.g., just for receiving announcements), and anytime you have external RTP to send you send it there. Suboptimal, of course (you need two PC instead of one) but solves your requirement without touching the code.

@lminiero
Copy link
Member

FYI, we recently added support for playing static Opus files in the AudioBridge in #2088, so that should already help with that you need. Actual support for incoming RTP streaming will take longer, as it's quite low in our priority list right now.

@lminiero
Copy link
Member

lminiero commented Dec 1, 2020

@austinmarkus please see #2464.

@austinmarkus
Copy link
Author

austinmarkus commented Dec 1, 2020 via email

@lminiero lminiero linked a pull request Dec 3, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants