Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#EXT-X-VERSION incorrect in examples and missing linefeeds #49

Open
piersoh opened this issue Apr 3, 2024 · 1 comment
Open

#EXT-X-VERSION incorrect in examples and missing linefeeds #49

piersoh opened this issue Apr 3, 2024 · 1 comment
Labels
bug Issue found with specification text or references discussion needed Further discussion required to resolve question
Milestone

Comments

@piersoh
Copy link

piersoh commented Apr 3, 2024

The examples in the existing DASH-HLS spec (CTA5005) that show Low Latency (Figs: 5.10, 5.11, 5.13) use #EXT-X-VERSION: 7 but it seems that should be #EXT-X-VERSION: 10 (or 9) since the low latency additions were only defined in draft-pantos-hls-rfc8216bis-07 which defines version 10 of HLS (perhaps there was a mix-up with the draft numbering?). It also seems that 9 is a possible version number for low latency as, despite it not being explicitly defined in a public spec, it is in examples on Apple's page on 'Enabling Low-Latency HTTP Live Streaming (HLS)': https://developer.apple.com/documentation/http-live-streaming/enabling-low-latency-http-live-streaming-hls#Examine-a-Low-Latency-HLS-Playlist

However if we want achieve backward-compatibility with playlists then the spec does mention in the Protocol Version Compatibility section that version 9 or above is only required if it contains the EXT-X-SKIP tag so version 7 or earlier (Apple uses version: 6 in one example) maybe ok?

Also in Figs 5.11 & 5.13 there's missing linefeed on the #EXT-X-VERSION line so it is joined with #EXT-X-PART-INF tag.

@technogeek00 technogeek00 added bug Issue found with specification text or references discussion needed Further discussion required to resolve question labels Apr 24, 2024
@technogeek00
Copy link
Collaborator

Update 2024/04/24

  • Reviewing it looks like #EXT-X-VERSION: 7 for the base variant playlist can be correct (6 would also be acceptable based on the features of the examples) as the low latency specific tags can be ignored by an incompatible player.
  • The playlist containing #EXT-X-SKIP is the only one required to be minimum version 9 (or 10) and will come with the delta update descriptions.
  • For the spec we should be consistent in the use of the lowest allowable version for HLS playlists which is consistent with recommendations.
  • Will get a double check from Apple on interpretation

@technogeek00 technogeek00 added this to the 3rd Edition milestone Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue found with specification text or references discussion needed Further discussion required to resolve question
Projects
None yet
Development

No branches or pull requests

2 participants