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

feat: HTTP API Authentication and Configuration Hot Reloading #44

Open
primemb opened this issue Aug 10, 2024 · 2 comments
Open

feat: HTTP API Authentication and Configuration Hot Reloading #44

primemb opened this issue Aug 10, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@primemb
Copy link

primemb commented Aug 10, 2024

Hi
Stream Sprout is an excellent solution for streaming to multiple destinations, providing functionality similar to services like Restream.io without the need for third-party services. However, I believe the project could be further improved with the addition of two key features:

HTTP API Authentication:

Currently, Stream Sprout does not offer built-in authentication for its HTTP API, which could potentially expose the API to unauthorized access if the service is hosted remotely.
Implementing authentication for the HTTP API would greatly enhance the security of Stream Sprout, especially when it is run on a remote server.
This could be achieved by introducing token-based authentication, basic auth, or even integrating with OAuth for more advanced use cases.
Configuration Hot Reloading:

At present, changes to the configuration file require a restart of the Stream Sprout service to take effect.
Introducing hot reloading of the YAML configuration file would allow users to update their stream destinations and other settings on the fly, without the need for a restart.
This feature would enhance the usability of Stream Sprout, making it more flexible and responsive to dynamic streaming needs.

@popey
Copy link
Collaborator

popey commented Aug 15, 2024

Hi @primemb, Thanks for the feature suggestions. These are both great ideas, but I suspect they are somewhat limited by the way stream-sprout is (currently) architected as a shell script wrapper around ffmpeg.

Some of these more advanced (and indeed very useful) features will require a rewrite in a 'proper' language, I think.

@popey popey added the enhancement New feature or request label Aug 15, 2024
@primemb
Copy link
Author

primemb commented Aug 16, 2024

well i think you can use ngnix rtmp instead of pure ffmpeg for RTMP server i used this docker before :

https://github.com/tiangolo/nginx-rtmp-docker

this also solve authentication problem and limitation that you already have

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