About the stream and parameters in the on_publish hook

[RocFang]

hi, @winlinvip

In the on_publish message of the http hook at 85d3a50, parameters are also included for the sent stream value. However, considering that the param has already been sent, is it necessary to repeat the parameters in the stream here?

Because considering that in the case where the parameter is only vhost=xxxx, there is no need to concatenate the stream name with the parameters, it feels a bit strange to have the stream as streamname?vhost=xxxx in on_publish.

TRANS_BY_GPT3

[RocFang]

For example, rtmp://127.0.0.1/live?vhost=a.com/test
In this format, the stream in the message sent by on_publish will also be concatenated as test?vhost=a.com.

TRANS_BY_GPT3

[RocFang]

This submission is for resolving.
a66d450

However, in srs2.0, there are two forms of parameters, either after the app or after the stream. If both cases exist, then req->param will be overwritten with the form of parameters after the stream with a question mark.

But now, regardless of whether the parameter form is after the app or after the stream, it will be appended to the stream when forwarding at the edge. The mentioned stream value in on_publish is also in this situation.

TRANS_BY_GPT3

[winlinvip]

SRS has been a pitfall on the URL for many years, and the root cause lies in my initial lack of understanding of the URI specification.

TRANS_BY_GPT3

[winlinvip]

Fixed in 6cea551

[winlinvip]

Fixed