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

No http api token authentication support for http-api #548

Closed
FlowerWrong opened this issue Dec 24, 2015 · 5 comments
Closed

No http api token authentication support for http-api #548

FlowerWrong opened this issue Dec 24, 2015 · 5 comments
Assignees
Labels
Enhancement Improvement or enhancement. Feature It's a new feature. TransByAI Translated by AI/GPT.
Milestone

Comments

@FlowerWrong
Copy link
Contributor

There is no token authentication support for http-api module.
I have found the srs/trunk/src/app/srs_app_http_api.cpp, but no code for this.

@winlinvip
Copy link
Member

Not support yet.

@winlinvip winlinvip added Enhancement Improvement or enhancement. Feature It's a new feature. labels Dec 24, 2015
@winlinvip winlinvip added this to the srs 3.0 release milestone Dec 24, 2015
@FlowerWrong
Copy link
Contributor Author

I also found that there is only security config for user who can push rtmp flow to srs server, but the ip of push flow client may change often. Can you support it on http-callback. For example, you can support it with on_publish event. The on_connect event can not do that.

@winlinvip
Copy link
Member

SRS does support on_publish and on_connect callback. The security is another solution, for example, some edge only allow play.

@FlowerWrong
Copy link
Contributor Author

Hi, I found that http callback api has a "tcUrl": "rtmp://x/x?key=xxx" in on_connect. But there is no one in on_publish, I want to deny someone to publish rtmp stream to srs server without username and password or named token.

@winlinvip
Copy link
Member

winlinvip commented Jan 8, 2016

OK, Accept.

TRANS_BY_GPT3

FlowerWrong added a commit to FlowerWrong/srs that referenced this issue Jan 8, 2016
winlinvip added a commit that referenced this issue Jan 9, 2016
Add tcUrl to on_publish http hook for issue #548
@winlinvip winlinvip self-assigned this Sep 24, 2021
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvement or enhancement. Feature It's a new feature. TransByAI Translated by AI/GPT.
Projects
None yet
Development

No branches or pull requests

2 participants