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

srs needs to support rtmp push address with parameters appended #1059

Closed
wnpllrzodiac opened this issue Feb 2, 2018 · 5 comments
Closed
Assignees
Labels
Enhancement Improvement or enhancement. Feature It's a new feature. TransByAI Translated by AI/GPT.
Milestone

Comments

@wnpllrzodiac
Copy link
Contributor

wnpllrzodiac commented Feb 2, 2018

The address format rtmp://127.0.0.1:1935/app?vhost=host1&param1=a&param2=b/stream123 is currently supported by SRS.
The address format rtmp://127.0.0.1:1935/app/stream123?vhost=host1&param1=a&param2=b is currently not supported.

However, common platforms like Alibaba Cloud use the latter format. It is hoped that both formats of RTMP streaming addresses can be supported.

TRANS_BY_GPT3

@juntaoliu
Copy link
Contributor

juntaoliu commented Feb 2, 2018

Parameters are easy to talk about, and supporting them is relatively simple.
But dealing with vhosts may be more troublesome.
In rtmp streaming, during the connect phase, only the path before stream123 is transmitted, and the part after stream123 is transmitted during the publish phase.
If the vhost is placed at the end, then the specific vhost may not be determined during the connect phase.

TRANS_BY_GPT3

@wnpllrzodiac
Copy link
Contributor Author

wnpllrzodiac commented Feb 2, 2018

I used nginx rtmp to create a redirect, combined with lua authentication, and added support for delayed backend parameters. The vhost is something that needs to be considered.

TRANS_BY_GPT3

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

winlinvip commented Feb 13, 2018

SRS2h supports this method, and the posterior approach is more in line with normal habits. Reference:

  1. https://github.com/ossrs/srs/wiki/v2_EN_RtmpUrlVhost#srs-url
  2. https://github.com/ossrs/srs/wiki/v2_CN_RtmpUrlVhost#srs-url

TRANS_BY_GPT3

@winlinvip
Copy link
Member

winlinvip commented Feb 13, 2018

Please test various URLs to ensure that they are all supported and check if any new bugs have been introduced.

Please help to test about all URLs. Please file issues if introduced new bug.

TRANS_BY_GPT3

@wuxianlijiang
Copy link

wuxianlijiang commented Feb 16, 2018

Thank you

TRANS_BY_GPT3

@winlinvip winlinvip self-assigned this Sep 12, 2021
@winlinvip winlinvip changed the title srs 需要支持参数后置的rtmp推流地址 srs needs to support rtmp push address with parameters appended 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

4 participants