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

Can the 'on_play' HTTP callback be enhanced to include the 'pageUrl'? #435

Closed
backylee opened this issue Jul 1, 2015 · 3 comments
Closed
Assignees
Labels
Enhancement Improvement or enhancement. Feature It's a new feature. TransByAI Translated by AI/GPT.
Milestone

Comments

@backylee
Copy link

backylee commented Jul 1, 2015

When publishing a stream, on_connect becomes on_publish.
When a client is playing, on_connect becomes on_play.

If you want to control client anti-leeching, the pageUrl in on_connect should not be used for whitelist checking, as it will affect the publishing process.
If you include pageUrl separately in on_play, you can perform the check.
Of course, SRS's Refer anti-leeching can also do this job, and if there is an HTTP callback, it will be more flexible.

TRANS_BY_GPT3

@winlinvip winlinvip added Enhancement Improvement or enhancement. Feature It's a new feature. labels Jul 2, 2015
@winlinvip winlinvip added this to the srs 2.0 release milestone Jul 2, 2015
@winlinvip
Copy link
Member

winlinvip commented Jul 2, 2015

I will add it in version 2.0.

TRANS_BY_GPT3

@winlinvip
Copy link
Member

@winlinvip
Copy link
Member

[2015-07-21 10:55:21][trace] srs on_play: client id=110, ip=127.0.0.1, 
vhost=__defaultVhost__, app=live, stream=livestream, 
pageUrl=http://www.ossrs.net/players/srs_player.html?vhost=dev&stream=livestream&server=dev&port=1935

[2015-07-21 10:55:35][trace] srs on_close: client id=110, ip=127.0.0.1, 
vhost=__defaultVhost__, app=live, 
send_bytes=634022, recv_bytes=3622

@winlinvip winlinvip self-assigned this Aug 26, 2021
@winlinvip winlinvip changed the title http回调on_play可否增加pageUrl Can the 'on_play' HTTP callback be enhanced to include the 'pageUrl'? Jul 28, 2023
@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