Skip to content

Conversation

@jinleileiking
Copy link

No description provided.

@davidzhao
Copy link
Member

@jinleileiking can you share more about why this is needed? In your slack thread, you mentioned srs. Does it not support echo verification?

@biglittlebigben
Copy link
Contributor

I'm afraid doing this by default may break compatibility with some other RTMP implementations. Would it be possible to make this behavior change behind a configuration variable?

@jinleileiking
Copy link
Author

jinleileiking commented Feb 21, 2023

I'm afraid doing this by default may break compatibility with some other RTMP implementations. Would it be possible to make this behavior change behind a configuration variable?

A configure is good.

@jinleileiking
Copy link
Author

@jinleileiking can you share more about why this is needed? In your slack thread, you mentioned srs. Does it not support echo verification?

srs: https://github.com/ossrs/srs

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@biglittlebigben
Copy link
Contributor

Closing this as stale

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants