This repository has been archived by the owner on Jan 12, 2019. It is now read-only.

Trim whitespace in playlist #282

Closed

forbesjo wants to merge 3 commits into videojs:master from forbesjo:test/whitespace

Contributor

forbesjo commented May 27, 2015

Fix from #279.

If playlists contain "\r\n" line endings or even whitespace lines the m3u8 parser falsely detects those lines as "empty" segment URI-s. This results in faulty duration calculation and possible failure at playback start. By trimming line data before feeding it into the parser this issue is fixed.

mikrohard and others added 3 commits

May 27, 2015 11:30


          Trim whitespace & newlines in line data before feeding it to the parser

6c71fa7


          Added test for whitespace in playlist

9a4caeb


          Moved whitespace removal into push()

aa03b8a

Member

dmlap commented May 27, 2015

LGTM

forbesjo closed this in

4f6eaeb

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet