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

Accept rtsp client without interleaved parameter #1511

Merged
merged 6 commits into from
Feb 18, 2025

Conversation

fmcloudconsulting
Copy link
Contributor

Fix #1510

@AlexxIT AlexxIT added the invalid This doesn't seem right label Dec 17, 2024
@AlexxIT
Copy link
Owner

AlexxIT commented Dec 17, 2024

Your changes are incorrect. The client can come for UDP transport.

@fmcloudconsulting
Copy link
Contributor Author

The existing code already expects TCP-only transport, so there is no regression with this PR?

@AlexxIT
Copy link
Owner

AlexxIT commented Dec 17, 2024

But it return 461 Unsupported transport error, so smart client can change UDP to TCP request. Many clients support this out of box.

@fmcloudconsulting
Copy link
Contributor Author

Got the point. Should be fixed now.

@AlexxIT
Copy link
Owner

AlexxIT commented Dec 17, 2024

It's wrong again. Some clients don't send unicast word. I think only first prefix should be enough.

@fmcloudconsulting
Copy link
Contributor Author

Fixed.
By the way, there was a typo between tr and transport variables, fixed!
I'll build & deploy on my side.

@AlexxIT AlexxIT removed the invalid This doesn't seem right label Dec 18, 2024
@AlexxIT AlexxIT self-assigned this Dec 18, 2024
@felipecrs
Copy link
Contributor

@fmcloudconsulting did the deploy on your side happened successfully? Is this PR working as expected?

@AlexxIT AlexxIT self-requested a review February 18, 2025 09:31
@AlexxIT AlexxIT merged commit 19cf781 into AlexxIT:master Feb 18, 2025
@AlexxIT
Copy link
Owner

AlexxIT commented Feb 18, 2025

Thanks!

@AlexxIT AlexxIT added this to the v1.9.9 milestone Feb 18, 2025
@fmcloudconsulting
Copy link
Contributor Author

Hello,

After hours of effort we were unable to get Genetec to work in RTSP over TCP.
Only RTSP over UDP seemed possible.
So we used mediamtx as restream server.

However, the patch proposed here has been tested for other clients and worked well.
Thanks for merging!

@AlexxIT
Copy link
Owner

AlexxIT commented Feb 19, 2025

It seems that the patch is useless because of the initially bad software.
I was responding in the issue that sw is not implementing the RTSP specification correctly.

@AlexxIT
Copy link
Owner

AlexxIT commented Mar 10, 2025

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.

Got Unsupported transport with Genetec
3 participants