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

add an on_publish_done directive #12

Open
parkervcp opened this issue Mar 28, 2021 · 2 comments
Open

add an on_publish_done directive #12

parkervcp opened this issue Mar 28, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@parkervcp
Copy link

To match the on_publish support it would be good to also send done messages. This would also match rtmp and would be a way to tag a stream as offline.

nginx-rtmp docs
https://github.com/arut/nginx-rtmp-module/wiki/Directives#on_publish_done

@iSchluff
Copy link
Member

I think the API is a bit problematic because it doesn't allow you to reliably know whether a stream is active or not. If either the relay or the api endpoint is restarted you can't really track the state anymore. It may be convenient but it's not really a good solution to the problem.

I think a better solution would be to poll the srtrelay api or nginx-rtmp api for that matter to see the currently active streams.

@parkervcp
Copy link
Author

parkervcp commented Mar 29, 2021

We are already using the api to get active streams but the same page we have for auth also tracks "live" status. It's something we had and I noticed all the keys that were authenticated showed as live. Looking at the nginx rtmp options I saw we were using the done directive.

@iSchluff iSchluff added the enhancement New feature or request label Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants