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

[v3] DRM: Re-create MediaKeys for each content on Philips' NETTV #1519

Merged
merged 1 commit into from
Aug 30, 2024

Conversation

peaBerberian
Copy link
Collaborator

This is a backport of #1515 for the v3

Fixes #1464

There's a new device (after LG and Panasonic TVs) which seems to have
issues when relying on the same `MediaKeys` instance on multiple
contents, which seems to be Philips's NETTV (both 2023 and 2024 devices
seem to have the issue as far as we know, and they advertise completely
different operating systems, so instead we identified the culprit as
"NETTV", which seems to be the web browser application Philips developed
for their TVs and which is present on web applications' user agent on
those devices).

So I disable the possibility for them here.
I still have to check if it works because I took as user-agent something
I found a little randomly on the web.

Note that disabling re-usage of the `MediaKeys` instance will mean no
in-memory `MediaKeySession` caching, which is a trick we use to both
speed up loading a previous content and preventing to much requests
on the license server also when doing that.

But we know most other players don't seem to do such tricks and the
speed up is not that much noticeable, so that shouldn't be a big issue.
@peaBerberian peaBerberian added bug This is an RxPlayer issue (unexpected result when comparing to the API) DRM Relative to DRM (EncryptedMediaExtensions) Compatibility Relative to the RxPlayer's compatibility with various devices and environments labels Aug 27, 2024
@peaBerberian peaBerberian added this to the 3.33.4 milestone Aug 27, 2024
@peaBerberian peaBerberian added the Priority: 0 (Very high) This issue or PR has a very high priority. Efforts should be concentrated on it first. label Aug 30, 2024
@peaBerberian peaBerberian merged commit bce7633 into legacy-v3 Aug 30, 2024
7 checks passed
@peaBerberian peaBerberian deleted the fix/titan-os-media-keys-v3 branch November 9, 2024 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This is an RxPlayer issue (unexpected result when comparing to the API) Compatibility Relative to the RxPlayer's compatibility with various devices and environments DRM Relative to DRM (EncryptedMediaExtensions) Priority: 0 (Very high) This issue or PR has a very high priority. Efforts should be concentrated on it first.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants