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

fix(DRM): keys missing in the license marked as decipherable #1429

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

Florent-Bouisset
Copy link
Collaborator

@Florent-Bouisset Florent-Bouisset commented Apr 11, 2024

Previously, even if certain keys were missing from a license, the representation was still marked as decipherable. This commit change this behavior to require explicit inclusion of keys in the license for the representation to be considered decipherable.

Relates to #1427

@peaBerberian
Copy link
Collaborator

peaBerberian commented Apr 23, 2024

So as we said (I'm just writing it here so we don't forget) maybe the best way would be to notice after the onKeyUpdate callback is called in the ContentDecryptor that the key id we asked for is not in there, and in that case blacklist that key id / init data.

We could potentially do that by linking a KeySessionRecord to the initial key id + init data, to check

@Florent-Bouisset Florent-Bouisset force-pushed the fix/issue-1427-missingKeysInLicense branch from 8dd39b3 to 5a2a115 Compare June 17, 2024 09:39
@peaBerberian peaBerberian added the Priority: 2 (Medium) This issue or PR has a medium priority. label Jun 17, 2024
…ing in license.

This affect the mode "singleLicensePer": "init-data". In this mode a key that was expected
to be in the lissing but that is missing should be considered as non usable.
This should improve the behavior when the player is configured as "singleLicensePer": "init-data",
and the license server provide proactively licenses for representation we did not intended to play
right now.
@Florent-Bouisset Florent-Bouisset force-pushed the fix/issue-1427-missingKeysInLicense branch from 5a2a115 to fa443d4 Compare June 19, 2024 09:54
@peaBerberian peaBerberian added this to the 4.2.0 milestone Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 2 (Medium) This issue or PR has a medium priority.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants