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.

Sign up for GitHub

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

Jump to bottom

Remove LHLS support #2864

Merged
merged 1 commit into from Jul 13, 2020
Merged

Remove LHLS support #2864

merged 1 commit into from Jul 13, 2020

Conversation

robwalch
Copy link
Collaborator

@robwalch robwalch commented Jul 2, 2020 •
edited

This PR will...

  • Remove parsing of EXT-X-PREFETCH and EXT-X-PREFETCH-DISCONTINUITY tags, and the fragment.prefetch flag
  • Default progressive streaming to false

Why is this Pull Request needed?

"LHLS" support deprecation. From #2861:

The LHLS spec has been abandoned in favor of Apple's Low-Latency HLS. I encourage anyone interested in chunked-transfer HLS streaming to review Apple's #EXT-X-PREFETCH-HINT guidelines recently covered at WWDC https://developer.apple.com/wwdc20/10229. Progressive streaming using fetch will be supported, but not enabled by default moving forward. Support for manifest tags in the deprecated LHLS spec will be removed. For those interested in the code used for streaming legacy LHLS chunked-transfer streams, the check out the feature/chunked-transfer-lhls branch.

Checklist

  • new unit / functional tests have been added (whenever applicable)
  • API or design changes are documented in API.md

@robwalch robwalch added this to In progress in Low-Latency HLS (LL-HLS) via automation Jul 2, 2020
@robwalch robwalch requested a review from itsjamie July 3, 2020 14:40
@robwalch robwalch mentioned this pull request Jul 3, 2020
Closed
3 tasks
@robwalch robwalch force-pushed the feature/remove-lhls-support branch 2 times, most recently from b37b6cd to a785a2c Compare July 6, 2020 18:25
@robwalch robwalch force-pushed the feature/remove-lhls-support branch from a785a2c to cc5fdfd Compare July 6, 2020 21:23
@robwalch robwalch merged commit c789bd8 into feature/v1.0.0 Jul 13, 2020
@robwalch robwalch deleted the feature/remove-lhls-support branch July 13, 2020 19:17
@robwalch robwalch moved this from In progress to Done in Low-Latency HLS (LL-HLS) Aug 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@itsjamie itsjamie Awaiting requested review from itsjamie

Assignees
No one assigned
Labels
None yet
Projects
Low-Latency HLS (LL-HLS)
Status: Done
Low-Latency HLS (LL-HLS)
  
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@robwalch