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 767769 - DASH MPD’s minimumUpdatePeriod in small letter will make parse fail
DASH MPD’s minimumUpdatePeriod in small letter will make parse fail
Status: RESOLVED INCOMPLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Windows
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-06-17 00:42 UTC by WeiChungChang
Modified: 2018-05-06 12:20 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description WeiChungChang 2016-06-17 00:42:56 UTC
Dear all:
 
I met a issued DASH live link & the root cause is it has a minimumUpdatePeriod by "PT5s".
 
Please be aware of the 's', it is case sensitive now in current parsing flow.
 
It means if we have a 'S' everything will be O.K but 's' fails instead.
 
I am not sure if ISO-23009-1 defined the format well but just see the format definition for xsd:duration as the links below.
 
http://books.xmlschemata.org/relaxng/ch19-77073.html
http://www.ostyn.com/standards/scorm/samples/ISOTimeForSCORM.htm 
 
They both say 's' is legal.
 
 
Besides capital letters, Should we support small letters also?
 
Thanks.
Comment 1 WeiChungChang 2016-06-17 00:46:23 UTC
Fixed my comment;

For xsd 's' is not allowed but
for ISO 8601 's' is legal.
Comment 2 Sebastian Dröge (slomo) 2016-06-17 08:25:36 UTC
The DASH specification mentions "xs:duration" as the type, so I would say streams with lowercase letters there are invalid.

Do you have actual streams that do it wrong? It might make sense to add support for that then. Would you like to provide a patch, plus a small unit test for it too?
Comment 3 Vivia Nikolaidou 2018-05-06 12:20:59 UTC
Closing this bug report as no further information has been provided. Please feel free to reopen this bug report if you can provide the information that was asked for in a previous comment.
Thanks!