Skip to content

fix(HLS): Fix playback of content with mixed containers (mp4 and ts) #8837

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

Merged
merged 1 commit into from
Jul 3, 2025

Conversation

avelad
Copy link
Member

@avelad avelad commented Jul 3, 2025

We currently have many integrated transmuxers, so we should avoid assigning the same mimetype to all unloaded variants.
We can dynamically transmux all content; we just need a compatible codec.
Partial reverts #4601

We currently have many integrated transmuxers, so we should avoid assigning the same mimetype to all unloaded variants. We can dynamically transmux all content; we just need a compatible codec.
Partial reverts shaka-project#4601
@avelad
Copy link
Member Author

avelad commented Jul 3, 2025

Example of why this fails:
Variants:

  1. MP4-AV1
  2. TS-H264 (before create the segment index, MP4-H264)

Load by default TS-H264, we don't want change the mimetype of the first one.

@avelad avelad requested a review from tykus160 July 3, 2025 12:44
@avelad avelad added type: bug Something isn't working correctly component: HLS The issue involves Apple's HLS manifest format priority: P1 Big impact or workaround impractical; resolve before feature release labels Jul 3, 2025
@avelad avelad added this to the v4.16 milestone Jul 3, 2025
@shaka-bot
Copy link
Collaborator

Incremental code coverage: No instrumented code was changed.

@avelad avelad merged commit f7c2295 into shaka-project:main Jul 3, 2025
33 of 37 checks passed
@avelad avelad deleted the hls-old-code branch July 3, 2025 13:18
avelad added a commit that referenced this pull request Jul 7, 2025
…8837)

We currently have many integrated transmuxers, so we should avoid
assigning the same mimetype to all unloaded variants.
We can dynamically transmux all content; we just need a compatible
codec.
Partial reverts #4601
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: HLS The issue involves Apple's HLS manifest format priority: P1 Big impact or workaround impractical; resolve before feature release type: bug Something isn't working correctly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants