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

minimumUpdatePeriod is not mandatory for live streams #69

Closed
joeyparrish opened this issue Apr 25, 2015 · 0 comments
Closed

minimumUpdatePeriod is not mandatory for live streams #69

joeyparrish opened this issue Apr 25, 2015 · 0 comments
Assignees
@joeyparrish
Labels
status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Milestone
v1.3.1

Comments

@joeyparrish
Copy link
Member

According to the spec, if minimumUpdatePeriod is not present, the MPD should not be updated.

We should be deciding that a stream is live based purely on type=='dynamic'.
@joeyparrish joeyparrish added the type: bug Something isn't working correctly label Apr 25, 2015
@joeyparrish joeyparrish self-assigned this Apr 25, 2015
@joeyparrish joeyparrish modified the milestones: v1.4.0, v1.3.1 Apr 25, 2015
joeyparrish added a commit that referenced this issue May 22, 2015
@joeyparrish
Choose live playback by type='dynamic'.
cae93b3 
Also avoid fetching manifest updates when minimumUpdatePeriod is
missing.  In the absence of minimumUpdatePeriod, just reprocess
the MPD we already downloaded.

Closes #69, #70.

Change-Id: I138bf136e5b0d691d9d625bbad02fb07ae21d32f
@shaka-project shaka-project locked and limited conversation to collaborators Mar 22, 2018
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Apr 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@joeyparrish joeyparrish

Labels
status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Milestone
v1.3.1
Development

No branches or pull requests
3 participants
@TheModMaker @joeyparrish @shaka-bot