After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 758921 - rtspsrc should use lowercase X-Sessioncookie header for compatibility with strict servers
rtspsrc should use lowercase X-Sessioncookie header for compatibility with st...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
1.6.1
Other All
: Normal normal
: 1.6.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-12-01 19:02 UTC by Evan Callaway
Modified: 2015-12-07 09:56 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Update capitalization of X-Sessioncookie to x-sessioncookie (1.47 KB, patch)
2015-12-02 14:12 UTC, Evan Callaway
committed Details | Review

Description Evan Callaway 2015-12-01 19:02:14 UTC
Consider a case where you are streaming RTSP over HTTP with a camera using the following pipeline:

gst-launch-1.0 rtspsrc location="rtsph://user:password@192.168.105.64:80/rtsp_tunnel" short-header=true ! decodebin ! fakesink dump=true

I found that the some Bosch cameras are unable locate my my X-Sessioncookie and the connection fails.  After debugging for some time, I found that the connection completes successfully if I change the capitalization of X-Sessioncookie to x-sessioncookie, which is a bit odd because I thought these headers are supposed to be capitalization agnostic.  The server must be doing a case-sensitive parse.

Quicktime and VLC both use the lowercase version "x-sessioncookie" and it is shown as lowercase in Apple's spec on how to do RTSP over HTTP (http://www.opensource.apple.com/source/QuickTimeStreamingServer/QuickTimeStreamingServer-412.42/Documentation/RTSP_Over_HTTP.pdf).  For compatibility with servers that are incorrectly strict, I suggest that we do the same...provided of course that this won't break other servers that expect it to be capitalized as we have it now.

Please let me know your thoughts on this issue.
Comment 1 Sebastian Dröge (slomo) 2015-12-02 07:16:56 UTC
If other software is doing the same, we should probably also do it. It's annoying but what can we do? :) Please also report this to Bosch though.

Do you want to provide a patch?
Comment 2 Evan Callaway 2015-12-02 14:12:29 UTC
Created attachment 316658 [details] [review]
Update capitalization of X-Sessioncookie to x-sessioncookie

Absolutely, here is a patch.  I have also reported this parsing quirk to Bosch.
Comment 3 Sebastian Dröge (slomo) 2015-12-02 14:30:55 UTC
commit e47643122c6d49b0633e53c5ea226fc898146f62
Author: Evan Callaway <evan.callaway@ipconfigure.com>
Date:   Wed Dec 2 09:00:31 2015 -0500

    rtspconnection: Update capitalization of x-sessioncookie
    
    Some servers incorrectly parse header names with strict case-sensitivity.  For
    compatibility with these systems change X-Sessioncookie to x-sessioncookie.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=758921