Skip to content

Accept rtsp client without interleaved parameter#1511

Merged
AlexxIT merged 6 commits intoAlexxIT:masterfrom
fmcloudconsulting:fix/rtsp-server-interleaved
Feb 18, 2025
Merged

Accept rtsp client without interleaved parameter#1511
AlexxIT merged 6 commits intoAlexxIT:masterfrom
fmcloudconsulting:fix/rtsp-server-interleaved

Conversation

@fmcloudconsulting
Copy link
Copy Markdown
Contributor

Fix #1510

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

AlexxIT commented Dec 17, 2024

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

@fmcloudconsulting
Copy link
Copy Markdown
Contributor Author

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

@AlexxIT
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

Got the point. Should be fixed now.

@AlexxIT
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Owner

AlexxIT commented Feb 18, 2025

Thanks!

@AlexxIT AlexxIT added this to the v1.9.9 milestone Feb 18, 2025
@fmcloudconsulting
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Owner

AlexxIT commented Mar 10, 2025

57Darling02 pushed a commit to 57Darling02/go2nvr that referenced this pull request Feb 17, 2026
…r-interleaved

Accept rtsp client without interleaved parameter
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