GNOME Bugzilla – Bug 715168
URL in RTP-Info miss the query part
Last modified: 2014-02-25 22:28:43 UTC
When responding to a PLAY request the query part of the URL is not included in the RTP-Info headers. From the RTSP v2.0 draft: url: Indicates the stream URI for which the following RTP parameters correspond, this URI MUST be the same as used in the SETUP request for this media stream. Any relative URI MUST use the Request-URI as base URI. This parameter MUST be present. I will attach a small patch fixing the issue.
Created attachment 261476 [details] [review] Include query part of request URL in RTP-Info header in PLAY response
If you want to implement if like that, you would need to store the URL used in the SETUP (in the session or so) and then use that to reply. You can't assume the query paramaters in the PLAY are the same as the ones used in SETUP.
Created attachment 262949 [details] [review] Save url's from SETUP request and use them when generating PLAY response
Changed it slightly to store the setup url in the StreamTransport, where the other config information is of the stream in the session. commit b16b47f68daccf0a939673f5246ed1d1ae88964b Author: Branko Subasic <branko@axis.com> Date: Wed Nov 27 15:04:04 2013 +0100 check: add test for uri in setup Added unit tests for the new functionality in GstRTSPStreamTransport. Fixes https://bugzilla.gnome.org/show_bug.cgi?id=715168 commit 421499c10274201d83c143627b6b487049e1192c Author: Wim Taymans <wtaymans@redhat.com> Date: Thu Nov 28 17:47:18 2013 +0100 client: store setup uri and use in PLAY response Store the uri used when doing the setup and use that in the PLAY response. fixes https://bugzilla.gnome.org/show_bug.cgi?id=715168 commit 568477d9b53c5dfb46b2cf1517a1885c3feb519a Author: Wim Taymans <wtaymans@redhat.com> Date: Thu Nov 28 17:35:45 2013 +0100 stream-transport: add method to get/set url