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

Poll Time for HLS Manifest #4771

Closed
dbanbahji opened this issue Nov 30, 2022 · 0 comments · Fixed by #4779
Closed

Poll Time for HLS Manifest #4771

dbanbahji opened this issue Nov 30, 2022 · 0 comments · Fixed by #4779
Assignees
Labels
component: HLS The issue involves Apple's HLS manifest format priority: P2 Smaller impact or easy workaround status: archived Archived and locked; will not be updated type: enhancement New feature or request
Milestone

Comments

@dbanbahji
Copy link

dbanbahji commented Nov 30, 2022

Have you read the FAQ and checked for duplicate open issues?
yes

Is your feature request related to a problem? Please describe.

yes. While earlier specs say to use the target duration tag to poll for manifest updates, later specs suggest the last segment duration as the poll time. Sometimes target duration is just a set length 1 or 2 seconds larger than the set segment length. So a manifest that has variable segment length which is very common in ad insertion workflows can sometimes have buffer starvation near the live edge. This should provide more robust support for hls ssai workflows. This is a spec that is followed commonly in commercial players like bitmovin, theoplayer, and OS hlsjs.

Describe the solution you'd like
use last downloaded segment duration as the poll time for the next manifest update. A variable poll time.
Basically later official spec versions describe that approach so it is spec approved.

When a client loads a Playlist file for the first time or reloads a
Playlist file and finds that it has changed since the last time it
was loaded, the client MUST wait for at least the duration of the
last segment in the Playlist before attempting to reload the Playlist
file again, measured from the last time the client began loading the
Playlist file.

https://datatracker.ietf.org/doc/html/draft-pantos-hls-rfc8216bis#section-6.3.4

Describe alternatives you've considered
n/a

Additional context
n/a

@dbanbahji dbanbahji added the type: enhancement New feature or request label Nov 30, 2022
@github-actions github-actions bot added this to the Backlog milestone Nov 30, 2022
@avelad avelad added component: HLS The issue involves Apple's HLS manifest format priority: P2 Smaller impact or easy workaround labels Dec 1, 2022
@avelad avelad self-assigned this Dec 1, 2022
@avelad avelad modified the milestones: Backlog, v4.4 Dec 1, 2022
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Feb 5, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: HLS The issue involves Apple's HLS manifest format priority: P2 Smaller impact or easy workaround status: archived Archived and locked; will not be updated type: enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants