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

HLS parser stops execution if it encounters a file format it can't parse the start time from #2108

Closed
ismena opened this issue Aug 20, 2019 · 4 comments
Assignees
Labels
component: HLS The issue involves Apple's HLS manifest format status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Milestone

Comments

@ismena
Copy link
Contributor

ismena commented Aug 20, 2019

Can you reproduce the issue with our latest release version?
Y

Can you reproduce the issue with the latest code from master?
Y

Are you using the demo app or your own custom app?
Demo

What did you do?

Load an hls manifest that has webm content.

What did you expect to happen?
It's either played or filtered out

What actually happened?
We're unable to parse the start time from webm segments and the execution stops.

FWIW, Wemb isn't an officially supported hls format and I've never seen webm+hls combination out there.
@joeyparrish can decide whether we want to support this, for now we should just skip the webm variants with a warning log and play what we can play.

@ismena ismena added type: bug Something isn't working correctly component: HLS The issue involves Apple's HLS manifest format labels Aug 20, 2019
@ismena
Copy link
Contributor Author

ismena commented Aug 20, 2019

Awesome @michellezhuogg volunteered to fix this!

@shaka-bot shaka-bot added this to the v2.6 milestone Aug 20, 2019
TheModMaker pushed a commit that referenced this issue Aug 22, 2019
We can't parse webm segments to extract segment start
time yet. Getting webm streams in hls stops playback
even if there are other formats we can play. This change
skips webm segments for now.

Fixes #2108

Change-Id: I4ff167efea45967cdf95a91afbe75efef3d9aa32
@joeyparrish joeyparrish reopened this Aug 26, 2019
@joeyparrish
Copy link
Member

The change as implemented was wrong. It doesn't skip WebM streams, it just assumes that they start at 0. The fix needs to be updated to actually ignore those streams and not generate variants for them.

@ismena
Copy link
Contributor Author

ismena commented Aug 26, 2019

Aw, that change did look to good to be true.
I'll fix the not-fixing fix.

@ismena ismena assigned ismena and unassigned michellezhuogg Aug 26, 2019
@joeyparrish
Copy link
Member

I believe this was fixed in 1ee3284.

@shaka-project shaka-project locked and limited conversation to collaborators Mar 23, 2020
@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.
Labels
component: HLS The issue involves Apple's HLS manifest format status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

No branches or pull requests

4 participants