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.

Sign up for GitHub

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

Jump to bottom

HlsParser SAMPLE-AES identity test failure #7152

Closed
joeyparrish opened this issue Aug 13, 2024 · 0 comments · Fixed by #7246
Closed

HlsParser SAMPLE-AES identity test failure #7152

joeyparrish opened this issue Aug 13, 2024 · 0 comments · Fixed by #7246
Labels
component: tests The issue involves our automated tests (generally; otherwise use a more specific component) type: bug Something isn't working correctly type: code health A code health issue
Milestone
v4.11

Comments

@joeyparrish
Copy link
Member

The test HlsParser supports SAMPLE-AES identity streaming has been failing on Edge and ChromecastUltra, possibly other devices as well.

1) supports SAMPLE-AES identity streaming
     HlsParser
     AbortError: The play() request was interrupted by a new load request.
     Error: Failed: audio queue should be empty after destroy!
    at <Jasmine>
    at Function.jasmineAssert (test/test/boot.js:35:7 <- test/test/boot.js:41:7)
    at _class._callee$ (lib/media/media_source_engine.js:438:22 <- lib/media/media_source_engine.js:18855:34)
    at tryCatch (node_modules/@babel/polyfill/dist/polyfill.js:6473:40)
    at Generator.invoke [as _invoke] (node_modules/@babel/polyfill/dist/polyfill.js:6702:22)
@joeyparrish joeyparrish added type: bug Something isn't working correctly type: code health A code health issue component: tests The issue involves our automated tests (generally; otherwise use a more specific component) labels Aug 13, 2024
@shaka-bot shaka-bot added this to the v4.11 milestone Aug 13, 2024
@avelad avelad mentioned this issue Aug 19, 2024
Merged
avelad added a commit that referenced this issue Aug 20, 2024
@avelad
test: Update SAMPLE-AES test asset (#7172)
416e169 
An asset with cenc is used instead of cbcs since cenc is compatible with
older versions of Tizen (our lab)

Related to #7152
joeyparrish pushed a commit that referenced this issue Aug 20, 2024
@avelad @joeyparrish
test: Update SAMPLE-AES test asset (#7172)
721ad3d 
An asset with cenc is used instead of cbcs since cenc is compatible with
older versions of Tizen (our lab)

Related to #7152
joeyparrish pushed a commit that referenced this issue Aug 20, 2024
@avelad @joeyparrish
test: Update SAMPLE-AES test asset (#7172)
bdc982c 
An asset with cenc is used instead of cbcs since cenc is compatible with
older versions of Tizen (our lab)

Related to #7152
@avelad avelad mentioned this issue Sep 2, 2024
Merged
@avelad avelad closed this as completed in 464b317 Sep 3, 2024
avelad added a commit that referenced this issue Sep 4, 2024
@avelad
test: Update SAMPLE-AES test asset (#7246)
9ea2f92 
It appears that playing a DRM muted audio-only asset fails on some
platforms.
Switching from an audio asset to a video asset.

Fixes #7152
avelad added a commit that referenced this issue Sep 4, 2024
@avelad
test: Update SAMPLE-AES test asset (#7246)
a062800 
It appears that playing a DRM muted audio-only asset fails on some
platforms.
Switching from an audio asset to a video asset.

Fixes #7152
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: tests The issue involves our automated tests (generally; otherwise use a more specific component) type: bug Something isn't working correctly type: code health A code health issue
Projects
None yet
Milestone
v4.11
Development

Successfully merging a pull request may close this issue.

test: Update SAMPLE-AES test asset avelad/shaka-player
2 participants
@joeyparrish @shaka-bot