-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add RTSP server #683
Comments
It has turned out that there is already an implementation of a simple RTSP server in Jellyfish. I slightly modified it and added it to the rtsp_to_hls demo: https://github.com/membraneframework/membrane_demo/blob/master/rtsp_to_hls/server.exs |
@varsill should we close this then? |
As discussed - closing since no further development is required at this moment. In the future, we might want to rewrite the RTSP server in such a manner, that each message could be responded to within a callback. |
Hi @varsill @mat-hek |
Hi, it looks good to me! Feel free to open a PR, thanks ;) |
Currently, the
membrane_rtsp
package (https://github.com/membraneframework/membrane_rtsp) contains only RTSP client, capable of receiving RTSP messages.Adding the corresponding RTSP server functionality (so that the user would be able to send RTSP messages) should be problematic and definitely it would be helpful, because we then could use such a server e.g. in our
rtsp_to_hls
demo.Apart from them, I believe the package name should be converted to
ex_rtsp
to follow our naming conventions.The text was updated successfully, but these errors were encountered: