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

Bugfix: Handle duplicate audio codec in CODECS #6341

Conversation

robwalch
Copy link
Collaborator

This PR will...

Avoid using "levelCodec" to instantiate the audio SourceBuffer when multiple audio codec strings are present in the Multivariant Playlist CODECS attribute for muxed content (TS segment without alt-audio).

Why is this Pull Request needed?

SourceBuffer creation fails with invalid mime-time.

Are there any points in the code the reviewer needs to double check?

Resolves issues:

Fixes #6337

Checklist

  • changes have been done against master branch, and PR does not conflict
  • new unit / functional tests have been added (whenever applicable)
  • API or design changes are documented in API.md

…io codec strings are present in the Multivariant Playlist variant CODECS attribute with muxed content

Fixes #6337
@robwalch robwalch added this to the 1.5.8 milestone Apr 10, 2024
@robwalch robwalch merged commit 380076a into patch/v1.5.x Apr 10, 2024
13 of 14 checks passed
@robwalch robwalch deleted the bugfix/sourcebuffer-creation-multiple-audio-codecs-in-manifest-with-ts branch April 10, 2024 21:21
robwalch added a commit that referenced this pull request Apr 11, 2024
* patch/v1.5.x:
  Bugfix: Patch for light player exception with audio groups (#6342)
  Do not use "levelCodec" to instantiate SourceBuffer when multiple audio codec strings are present in the Multivariant Playlist variant CODECS attribute with muxed content (#6341)
  Bugfix: preferManagedMediaSource:false when MSE is not present  (#6338)
  Determine canSkip based on age of last playlist request (#6300)
  API.md update removeLevel
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

Successfully merging this pull request may close these issues.

1 participant