[gstreamer-bugs] [Bug 586180] New: RTSP improvements

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=586180

  GStreamer | gst-plugins-base | Ver: git
           Summary: RTSP improvements
           Product: GStreamer
           Version: git
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-base
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: pkj at axis.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


As requested by Wim I add a report with my suggested improvements to the RTSP
code. The original discussion concerned commit 8ed062f (support for HTTP
messages), but I have included all my intended changes here for review as I
believe it will better show the ratinale for adding simple HTTP support to the
RTSP code.

The patch contains the following commits:

e326849... rtsp: Add initial buffer support.
c6de461... rtsp: Added gst_rtsp_connection_create_from_fd().
8ed062f... rtsp: Added support for HTTP messages
d0b5b51... rtsp: Improved parsing of messages.
b97d6c7... rtsp: Made parse_string() return a result.
9ae3686... rtsp: Normalize lines (remove extra whitespace) before parsing.
a033959... rtsp: Ignore the Content-Length for POST requests.
b32db61... rtsp: Rewrote gen_tunnel_reply().
7784a77... rtsp: Rewrote setup_tunneling().
b052009... rtsp: Improve parse_line().
27345f4... rtsp: Add error_full callback to GstRTSPWatchFuncs.
ebfb315... rtsp: Add message_blocked to GstRTSPWatchFuncs.
1a946fc... rtsp: Write as much as possible in gst_rtsp_source_dispatch().
a21e495... rtsp: Allow gst_rtsp_connection_do_tunnel() to just setup decoding
co
ntext.
6fc6010... rtsp: Added gst_rtsp_connection_set_http_mode().

I have also provided the final gstrtspconnection.c file as it can sometimes be
hard to tell from the diffs what's actually going on.

I have tried my utmost to main compatibility with the current code (including
what I believe to be a number of bugs in setup_tunneling()), but as I only use
the code for an RTSP server the code is only partially tested for client
support.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=586180.