[Bug 710223] New: [smoothstreaming] Strip both /manifest and /Manifest from the manifest uri

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Wed Oct 16 00:42:30 CEST 2013


https://bugzilla.gnome.org/show_bug.cgi?id=710223
  GStreamer | gst-plugins-bad | 1.2.0

           Summary: [smoothstreaming] Strip both /manifest and /Manifest
                    from the manifest uri
    Classification: Platform
           Product: GStreamer
           Version: 1.2.0
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: minor
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: ospite at studenti.unina.it
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


When building mssdemux->base_url in
gstmssdemux.c::gst_mss_demux_process_manifest() strip both /manifest and
/Manifest (i.e. ignoring the case) to be more robust against copy/paste.

My use case is that I paste the URI I copied from inside a composite manifest
like this one:
http://originsmoothreplay.rai.it/csm_nrp/raidue/Made_in_Sud/madeinsud_puntate_2012/1891183.csm

You can see that the Url attribute of the Clip elements ends with a lower case
"/manifest" string.

Maybe the code to strip the last part of the path when building the base_url
could use regexes, in order to be more easily generalized later to strip also
the names of composites manifests (e.g. /1891183.csm) when support for those
are added.

Thanks, Antonio

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