New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 902828 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Relax session ID restrictions

Project Member Reported by yucliu@chromium.org, Nov 7

Issue description

Some CDMs (e.g. PlayReady) generate session IDs in the format of base64 encoded. Currently Chromium only allows ASCII session IDs.

https://cs.chromium.org/chromium/src/third_party/blink/renderer/modules/encryptedmedia/media_key_session.cc?q=media_key_session.cc&dr&l=81

https://cs.chromium.org/chromium/src/media/blink/webcontentdecryptionmodulesession_impl.cc?type=cs&q=+media/blink/webcontentdecryptionmodulesession_impl.cc&sq=package:chromium&g=0&l=156

Based on EME spec:

"A Session ID is a unique string identifier generated by the CDM that can be used by the application to identify MediaKeySession objects."

(https://www.w3.org/TR/encrypted-media/#session-id)

"The user agent should thoroughly validate the sessionId value before passing it to the CDM. At a minimum, this should include checking that the length and value are reasonable (e.g., not longer than tens of characters and alphanumeric)."

(https://www.w3.org/TR/encrypted-media/#dom-mediakeysession-load)

We should either allow base64 encoded session IDs, or ask the key system to validate the session IDs.


 
Status: Assigned (was: Untriaged)
This issue has an owner, a component and a priority, but is still listed as untriaged or unconfirmed. By definition, this bug is triaged. Changing status to "assigned". Please reach out to me if you disagree with how I've done this.

Sign in to add a comment