[Bug 736267] New: rtspclient: Be more forgiving when parsing session header in requests

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736267
  GStreamer | gst-plugins-base | git

           Summary: rtspclient: Be more forgiving when parsing session
                    header in requests
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-base
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: ognyan.tonchev at axis.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Created an attachment (id=285650)
 View: https://bugzilla.gnome.org/attachment.cgi?id=285650
 Review: https://bugzilla.gnome.org/review?bug=736267&attachment=285650

rtspconnection: ignore timeout in session request header

According to the rfc the timeout parameter is only allowed in a session
response header but some clients, like Honeywell VMS applications, send it as
part of a session request header too:

PLAY
rtsp://172.26.151.197/axis-media/media.amp?videocodec=h264&camera=1&videokeyframeinterval=8&resolution=1280x720&compression=41&fps=60&color=1
RTSP/1.0
CSeq: 3
User-Agent: Axis AMC
Session: fNpBsc72MwWfp%2BsL; timeout=60

Would you accept a patch which basically ignores everything from the semicolon
to the end of the line when parsing session ids? Or perhaps checking for any
non-allowed character?

Currently rtspconnection takes everything from the colon as session id.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.