-
Notifications
You must be signed in to change notification settings - Fork 182
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
RTP-Info header is missing #43
Comments
Hi. I'm having a look at the code and the RFC. Are you happy to re-test as I make the changes? Thanks |
Yes, that would be great. You are right about the Range header, it also needs to be present in the PLAY response, to be used together with the RTP-Info header.
Please let me know when you make these changes, I will give them a try. (No earlier than August 13, though, as I will be on vacation until then.)
Thanks,
Anders
…________________________________
From: Roger Hardiman <[email protected]>
Sent: Tuesday, July 31, 2018 12:29:27 AM
To: ngraziano/SharpRTSP
Cc: Anders Klemets; Author
Subject: Re: [ngraziano/SharpRTSP] RTP-Info header is missing (#43)
Hi.
Thanks for the bug reports.
The RTSPCameraServer was originally a quick proof of concept. I published the code and then forgot about it (I use the library mainly as a client)
I'm having a look at the code and the RFC.
I will either add RTP-Info with just the URL and the Sequence Number, or I'll drop in Range: npt=xxxx- so there is a start time to go with the RTP-Info RtpTimestamp
Are you happy to re-test as I make the changes?
Thanks
Roger
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#43 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/Aaa3gsuk8cU1ij1wAenQ7F9YKcmziYQAks5uMAdXgaJpZM4VneBz>.
|
First set of fixes done. SSRC is now in Transport. |
Extra information emailed in by Andres... I am back from vacation and pulled down the fix that you made to the RTSP server code to advertise the ssrc in the RTSP SETUP response. I tested my client stack against it, and this part is working fine. I notice a couple of other problems, however:
If you fix any of these issues, I will be happy to test again. |
I'm running Multiple RTSP servers on a single machine (far from optimal) on different ports to provide multiple camera streams, also noticed the client-ports in the UDP transport response were strange. plus the sdp was missing some info: |
Hi @ikriz If you could open a new Issue report about the UDP ports, that would be great. But for now I'll ask about the "a=control:xxxxx" issue. RFC 2326 Section C.1.1 tells us that In your post you have swapped this to an Absolute URL. Then I can understand why the Absolute URL worked when the relative one did not. Thanks |
The RTSP PLAY response does not include the RTP-Info header. This header includes the initial RTP time stamp and sequence number. It is important, the absence of this header can cause interop problems.
Also the Transport header in the SETUP response should include the ssrc parameter (which specifies the SSRC of the RTP packets.)
The text was updated successfully, but these errors were encountered: