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

Add support to <dashif:Laurl> #4748

Closed
0xAllDe opened this issue Nov 23, 2022 · 2 comments · Fixed by #4849
Closed

Add support to <dashif:Laurl> #4748

0xAllDe opened this issue Nov 23, 2022 · 2 comments · Fixed by #4849
Assignees
Labels
component: DASH The issue involves the MPEG DASH manifest format component: EME The issue involves the Encrypted Media Extensions web API priority: P3 Useful but not urgent status: archived Archived and locked; will not be updated type: enhancement New feature or request
Milestone

Comments

@0xAllDe
Copy link

0xAllDe commented Nov 23, 2022

Have you read the Tutorials?
Yes

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

What version of Shaka Player are you using?
4.3

Please ask your question
I trying to play clear key licensing server using dashif:laurl but shaka-player return 6012 error.
I know Dash-IF lastest security guidelines is <dashif:laurl>
https://dashif-documents.azurewebsites.net/Guidelines-Security/master/Guidelines-Security.html#CPS-mpd-drm-config

I wondering, is shaka-player sopport <dashif:laurl>? If it is, why I receive 6012 error?

tested <ContentProtection> Element like this

<ContentProtection schemeIdUri="urn:uuid:1077efec-c0b2-4d02-ace3-3c1e52e2fb4b" value="ClearKey1.0">
        <cenc:pssh>AAAANHBzc2gBAAAAEHfv7MCyTQKs4zweUuL7SwAAAAEbdF33h/WzyyI+2iPFMZsxAAAAAA==</cenc:pssh>
        <dashif:Authzurl>https://some.com/authorize</dashif:Authzurl>
        <dashif:laurl>http://licenseserver.com/AcquireLicense</dashif:laurl>
</ContentProtection>

Thanks to read my question.

2022-11-23 150110

@0xAllDe 0xAllDe added the type: question A question from the community label Nov 23, 2022
@joeyparrish
Copy link
Member

No, not yet, but it could be added. We support the older MS PlayReady variant of that already ms:laurl, so there's no reason we couldn't support dash:laurl as well.

I'll convert this into a feature request. PRs are welcome!

@joeyparrish joeyparrish added type: enhancement New feature or request component: EME The issue involves the Encrypted Media Extensions web API priority: P3 Useful but not urgent component: DASH The issue involves the MPEG DASH manifest format flag: seeking PR We are actively seeking PRs for this; we do not currently expect the core team will resolve this flag: good first issue This might be a relatively easy issue; good for new contributors and removed type: question A question from the community labels Dec 8, 2022
@avelad
Copy link
Member

avelad commented Dec 8, 2022

@bttop83 if you share a manifest(url), I can work in this feature. Thanks!

@github-actions github-actions bot added this to the Backlog milestone Dec 8, 2022
@avelad avelad changed the title is <dash:laurl> support? Add support to <dashif:Laurl> Dec 20, 2022
@avelad avelad self-assigned this Dec 20, 2022
@avelad avelad modified the milestones: Backlog, v4.4 Dec 20, 2022
@avelad avelad removed flag: seeking PR We are actively seeking PRs for this; we do not currently expect the core team will resolve this flag: good first issue This might be a relatively easy issue; good for new contributors labels Dec 20, 2022
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Apr 1, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: DASH The issue involves the MPEG DASH manifest format component: EME The issue involves the Encrypted Media Extensions web API priority: P3 Useful but not urgent status: archived Archived and locked; will not be updated type: enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants