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 710223 - smoothstreaming: strip both /manifest and /Manifest from the manifest uri
smoothstreaming: strip both /manifest and /Manifest from the manifest uri
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.2.0
Other Linux
: Normal minor
: 1.2.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-10-15 22:42 UTC by Antonio Ospite
Modified: 2013-11-16 17:10 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Antonio Ospite 2013-10-15 22:42:27 UTC
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
Comment 1 Thiago Sousa Santos 2013-11-15 15:31:16 UTC
commit fe77ee47c957a563dd879387eae902deefecf86f
Author: Thiago Santos <ts.santos@partner.samsung.com>
Date:   Mon Nov 11 16:35:35 2013 -0300

    mssdemux: also consider /manifest
    
    The spec says /Manifest, but /manifest is easily found on some
    services, so handle that, too.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=710223


This commit fixes the Manifest / manifest issue. For the CSM files support, it seems to me (from a quick reading on the internet) that it is more of a playlist than a media itself. Do you have any other stream examples? Is there any open info online other than http://msdn.microsoft.com/en-us/library/ff402314%28v=vs.95%29.aspx ?

I'm closing this bug as the main issue has been fixed. Would you mind opening a new one to discuss the composite manifests implementation in case you are interested in them?
Comment 2 Antonio Ospite 2013-11-16 17:10:51 UTC
Thanks Thiago, the report about the composite manifest is bug 710221