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

(HLS, TS, AES-128 key rotation) occur 3014, 3016 error in Android Chrome. #6334

Closed
papico-inflab opened this issue Mar 7, 2024 · 10 comments
Closed
Assignees
Labels
platform: Android Issues affecting Android priority: P2 Smaller impact or easy workaround status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Milestone

Comments

@papico-inflab
Copy link

Have you read the FAQ and checked for duplicate open issues?
yes

If the problem is related to FairPlay, have you read the tutorial?

yes

What version of Shaka Player are you using?

v4.7.11

Can you reproduce the issue with our latest release version?
yes

Can you reproduce the issue with the latest code from main?
yes

Are you using the demo app or your own custom app?
demo app

What browser and OS are you using?
browser: Chrome (122.0.6261.91)
OS: Android 13

What are the manifest and license server URIs?

https://storage.googleapis.com/shaka-demo-assets/sintel-ts-aes-key-rotation/master.m3u8

Demo site url : https://shaka-player-demo.appspot.com/demo/#audiolang=en-US;textlang=en-US;uilang=en-US;asset=https://storage.googleapis.com/shaka-demo-assets/sintel-ts-aes-key-rotation/master.m3u8;panel=ALL_CONTENT;panelData=drm:AES128,HLS,MP2TS;build=compiled

What did you do?

Enter the link on Android device Chrome

What actually happened?

Error 3016, 3014 when playing with AES-128 key rotation on Android Chrome browser.

CleanShot 2024-03-07 at 17 17 02@2x
Screenshot_20240304_145542_Chrome

@papico-inflab papico-inflab added the type: bug Something isn't working correctly label Mar 7, 2024
@shaka-bot shaka-bot added this to the v5.0 milestone Mar 7, 2024
@joeyparrish joeyparrish added priority: P2 Smaller impact or easy workaround platform: Android Issues affecting Android labels Mar 7, 2024
@joeyparrish
Copy link
Member

Confirmed. It works in ChromeOS and Chrome on Linux, but not Chrome on Android.

We need to check if there is a difference in TS support (is Android transmuxing like Linux and ChromeOS?), and we need to check if webCrypto is operating correctly on Android (unlikely to be the issue, but you never know)

@joeyparrish
Copy link
Member

My phone appears to support TS natively, which means we are skipping transmuxing. I didn't realize that was the case.

@avelad
Copy link
Member

avelad commented Mar 7, 2024

@joeyparrish I think TS has problems in all browsers, I think we should always do transmuxing in any browser

@joeyparrish
Copy link
Member

Agreed. It comes up on TVs, on Android, on Cast... it's a bad format and it's badly-supported. Transmuxing should be better.

@papico-inflab
Copy link
Author

@joeyparrish
Any idea when this error might be fixed?
I'm currently playing with rotation aes-128 key encryption and that error is preventing me from updating the version.
Thank you for your attention.

@joeyparrish joeyparrish self-assigned this Mar 12, 2024
@joeyparrish
Copy link
Member

To fix this, we need to force transmuxing on all TS content on all platforms. That will require a new release.

But you can change this behavior with a config today:

// v4.4 through v4.7 (current)
player.configure('mediaSource.forceTransmux', true);

// v4.3
player.configure('streaming.forceTransmux', true);

// v3.0 through v4.2
player.configure('streaming.forceTransmuxTS', true);

Based on this workaround, the priority for the change is set at P2. Apologies for not explaining this workaround explicitly before. @papico-inf, can you please confirm that this workaround works for you?

@framework20
Copy link

framework20 commented Mar 13, 2024

Same errors - 3014 & 3016 on android (samsung s10+, android 12) with chrome (122.0.6261.105). But error 3014 does not always appear and has data[0]=0. Error 3016 has the following data [3,null, "PIPELINE_ERROR_DECODE"]. Logs from chrome://media-internals attached as a file. Shaka player v4.7.11. Firefox on android and desktop, chrome on desktop works fine. By the way, for some reason chrome on windows 11 and firefox on android allow me to take screenshots.
media-internals.txt

@papico-inflab
Copy link
Author

@joeyparrish
I tried setting the options you mentioned, but it doesn't work. 😢

avelad added a commit that referenced this issue Apr 3, 2024
Related to #6334
Related to
#6320 (comment)
Also reverts #6045
since now it is not necessary
@avelad
Copy link
Member

avelad commented Apr 5, 2024

@papico-inf can you test if the https://nightly-dot-shaka-player-demo.appspot.com/ fix the issue? Thanks!

@avelad avelad added the status: waiting on response Waiting on a response from the reporter(s) of the issue label Apr 5, 2024
avelad added a commit that referenced this issue Apr 8, 2024
Related to #6334
Related to
#6320 (comment)
Also reverts #6045
since now it is not necessary
avelad added a commit that referenced this issue Apr 8, 2024
Related to #6334
Related to
#6320 (comment)
Also reverts #6045
since now it is not necessary
@shaka-bot
Copy link
Collaborator

Closing due to inactivity. If this is still an issue for you or if you have further questions, the OP can ask shaka-bot to reopen it by including @shaka-bot reopen in a comment.

@shaka-bot shaka-bot removed the status: waiting on response Waiting on a response from the reporter(s) of the issue label Apr 12, 2024
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Jun 11, 2024
@shaka-project shaka-project locked as resolved and limited conversation to collaborators Jun 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
platform: Android Issues affecting Android priority: P2 Smaller impact or easy workaround status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

No branches or pull requests

5 participants