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

Parse bugs #358

Closed
anylayer opened this issue Mar 5, 2024 · 7 comments
Closed

Parse bugs #358

anylayer opened this issue Mar 5, 2024 · 7 comments

Comments

@anylayer
Copy link

anylayer commented Mar 5, 2024

https://m3u.haiwaikan.com/xm3u8/fff177dd1a7711a5b9647cb879c612b5832453bee2ab044c40504e42fded5e429921f11e97d0da21.m3u8

The first frame ts parsing failed and was ignored

Version: 4.0.0

@mauricioabreu
Copy link
Member

@anylayer can you reproduce the bug using the version from master branch?
I tried it locally using the latest commit and it worked.

>>> m3u8.load("test.m3u8")
<m3u8.model.M3U8 object at 0x100e57cd0>

@anylayer
Copy link
Author

@anylayer
Copy link
Author

https://m3u.haiwaikan.com/xm3u8/fff177dd1a7711a5b9647cb879c612b5832453bee2ab044c40504e42fded5e429921f11e97d0da21.m3u8
It seems not to be working
version: 4.1.0

image image

The first ts file of this m3u8 doesn't seem to have been parsed.

@mauricioabreu
Copy link
Member

Thank you for the info!

@mauricioabreu mauricioabreu added bug and removed bug labels May 11, 2024
@mauricioabreu
Copy link
Member

The problem is your playlist.

This is the beginning of your playlist:

#EXTM3U
#EXT-X-VERSION:3
#EXT-X-MEDIA-SEQUENCE:0
#EXT-X-ALLOW-CACHE:YES
#EXT-X-TARGETDURATION:19
https://cdn.iz8qkg.com/fvod/fa0c5b4bbc45b8be40444db98f559c5ac7c7f0eb58e4b4ee1e6e28dd3568af5facf6b6fc65342a15fe97b1b115ba2e365534c7f60de7ec6d689572d763b07fc94b06a343b93d62c2921c07e8bfad678982f74bbb85bcca4b.ts

As you can see, the first segment is malformed because it is missing #EXTINF tag.

Closing since it is not a m3u8 bug. Reopen if needed.

@leandromoreira
Copy link
Contributor

@davigsousa is working on a better error handling, that would provide better dev exp :) 🤞

cc @mauricioabreu @anylayer

@leandromoreira
Copy link
Contributor

#341

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants