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

feat: parse mp4protection ContentProtection tags #179

Merged
merged 2 commits into from
Dec 8, 2023

Conversation

adrums86
Copy link
Contributor

@adrums86 adrums86 commented Dec 8, 2023

Description

In an effort to enhance our HDCP handling in DASH we need to parse ContentProtection tags with the scheme urn:mpeg:dash:mp4protection:2011. These tags when defined with the urn:mpeg:cenc:2013 namespace contain a cenc:default_KID attribute that we can use to determine from the MediaKeySession, which playlists we have a usable license for.

See: https://dashif-documents.azurewebsites.net/Guidelines-Security/master/Guidelines-Security.html#CPS-default_KID
or
Reference: ISO_IEC 23009-1_2022 section 5.8.5.2

Changes

  • Added the urn:mpeg:dash:mp4protection:2011 schemeId to the keySystems map so we parse ContentProtection tags with that scheme.
  • Fixed tests.

Copy link

codecov bot commented Dec 8, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (56eee17) 94.68% compared to head (d771b45) 93.61%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #179      +/-   ##
==========================================
- Coverage   94.68%   93.61%   -1.07%     
==========================================
  Files          19       19              
  Lines         846      846              
  Branches      258      258              
==========================================
- Hits          801      792       -9     
- Misses         45       54       +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@adrums86 adrums86 merged commit 0813929 into main Dec 8, 2023
7 of 8 checks passed
@adrums86 adrums86 deleted the feat-parse-default_KID branch December 8, 2023 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants