[Bug 675624] [PATCH] [souphttpsrc] Add new features required to support DASH streaming

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Thu Dec 6 03:46:19 PST 2012


https://bugzilla.gnome.org/show_bug.cgi?id=675624
  GStreamer | gst-plugins-good | 0.10.36

Julien MOUTTE <julien> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |julien at moutte.net

--- Comment #1 from Julien MOUTTE <julien at moutte.net> 2012-12-06 11:46:16 UTC ---
You really don't need all those changes on souphttpsrc to implement a DASH
element in GStreamer.

- Partial get requests can be handled with extra headers using range requests.
- Persistent connections pool is handled by libsoup at a lower level already.
- Buffering should be handled in downstream elements
- Error handling to try alternative URLs should also be handled by the
application or the bin using souphttpsrc.
- Queueing the next segment in souphttpsrc can be done by the application or
the managing bin.

I think that trying to implement all this logic in souphttpsrc goes against the
classical design of GStreamer elements, souphttpsrc is just an HTTP downloader,
the logic for DASH and other protocols should be implemented by the app/bin
that is using this source.

This is the way we use at Fluendo for our DASH demuxer and we've been able to
make a fully functional demuxer without patching a single line of code in
souphttpsrc.

Best regards,

-- 
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.


More information about the gstreamer-bugs mailing list