iheartradio / open-m3u8

Open Source m3u8 Parser
Other
245 stars 94 forks source link

Comma after extinf tag #44

Closed saravindpsg closed 6 years ago

saravindpsg commented 7 years ago

Add comma separator at the end of #EXTINF tag line even if title is not present.

Without the comma separator, I found out that HLS Playback fails for some players which expect a comma at the end of EXTINF tag.

Here is the link to the HLS spec:

https://tools.ietf.org/html/draft-pantos-http-live-streaming-20#section-4.3.2.1

Wopple commented 6 years ago

I am surprised but you are correct. The spec does indeed say a comma is required.

I think the parsing should still be permissive because:

LGTM