Skip to content

fix(HLS): Fix preload initial variant #8835

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 5 commits into from
Jul 7, 2025

Conversation

avelad
Copy link
Member

@avelad avelad commented Jul 3, 2025

This is necessary to obtain the necessary DRM information, which in most cases is not available until the media playlist is downloaded.
Also avoid DRM setup for VOD that does not need it for HLS.

Fixes #8830

This is necessary to obtain the necessary DRM information, which in most cases is not available until the media playlist is downloaded.
Also avoid DRM setup for VOD that does not need it for HLS

Fixes shaka-project#8830
@avelad avelad marked this pull request as draft July 3, 2025 10:57
@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

shaka-bot commented Jul 3, 2025

Incremental code coverage: 97.59%

@avelad avelad marked this pull request as ready for review July 3, 2025 13:53
@avelad avelad requested review from tykus160, joeyparrish and TAhub July 3, 2025 13:53
@avelad avelad requested a review from tykus160 July 7, 2025 08:43
@avelad avelad merged commit c6b1390 into shaka-project:main Jul 7, 2025
45 of 48 checks passed
@avelad avelad deleted the preload-variant branch July 7, 2025 09:50
avelad added a commit that referenced this pull request Jul 7, 2025
This is necessary to obtain the necessary DRM information, which in most
cases is not available until the media playlist is downloaded.
Also avoid DRM setup for VOD that does not need it for HLS.

Fixes #8830
avelad added a commit to avelad/shaka-player that referenced this pull request Jul 15, 2025
Regression introduced in shaka-project#8835 by changing the order of operations in the preload manager.
avelad added a commit that referenced this pull request Jul 15, 2025
Regression introduced in
#8835 by changing the
order of operations in the preload manager.
avelad added a commit that referenced this pull request Jul 21, 2025
Regression introduced in
#8835 by changing the
order of operations in the preload manager.
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.

HLS + cbcs w/ Widevine playback fails without error
4 participants