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

Use BaseUri as the request Url when making the GET_PARAMETER request #12

Open
wants to merge 4 commits into
base: dev-v2-rtsp
Choose a base branch
from

Conversation

Willieodwyer
Copy link

This ensures that the Url used is the one returned from the DESCRIBE request

This was the previous behaviour when speaking to a Live555 RTSP Server

Request:
V/Sender: DESCRIBE rtsp://user:[email protected]:554/video?input=0&transmission=unicast RTSP/1.0 uri="rtsp://user:[email protected]:554/video?input=0&transmission=unicast"
Response:
V/Receiver: RTSP/1.0 200 OK Content-Base: rtsp://127.0.0.1/video?input=0&profile=primary&transmission=unicast/

after some time streaming

Request:
V/Sender: GET_PARAMETER rtsp://user:[email protected]:554/video?input=0&transmission=unicast RTSP/1.0 uri="rtsp://user:[email protected]:554/video?input=0&transmission=unicast"
Response:
V/Receiver: RTSP/1.0 404 Not Found

As you can see rtsp://127.0.0.1/video?input=0&transmission=unicast/ is missing the profile=primary part of the request string resulting in a 404

… request

This ensures that the Url used is the one returned from the DESCRIBE request
…ervers

Wait for dispatcher to finish requests before closing, so that TEARDOWN is complete
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.

1 participant