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
docs: update fairplay.md
#3562
docs: update fairplay.md
#3562
Conversation
Hi @joeyparrish, this PR is ready for review. |
docs/tutorials/fairplay.md
Outdated
|
||
## DRM Engine | ||
|
||
Some FairPlay support needs to be detected in order to change source to HLS if FairPlay is supported (AFAIK currently FairPlay is supported only within HLS and not publicly supported in DASH). You can query EME yourself, directly, and only query FairPlay, rather than the full list of key systems probed by `DrmEngine`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Split lines at 80 characters. Also, there's no need to mention DrmEngine
; you could say probeSupport
instead.
Some FairPlay support needs to be detected in order to change source to HLS if FairPlay is supported (AFAIK currently FairPlay is supported only within HLS and not publicly supported in DASH). You can query EME yourself, directly, and only query FairPlay, rather than the full list of key systems probed by `DrmEngine`. | ||
|
||
```js | ||
async function isFairPlaySupported() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It may be a good idea to add this to shaka.util.FairPlayUtils
so apps can just call this function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should I go ahead and add it to fairplay_utils.js?
@TheModMaker Please review the changes
This adds 3 new methods to FairPlayUtils: - isFairPlaySupported --> Closes PR #3562 - spcFairPlayRequest --> This method adds basic spc FairPlay request filter - commonFairPlayResponse --> This method adds a basic FairPlay response filter that supports a lot of DRM providers.
#4029 appears to make most of this obsolete. Please update the PR or close it. Thanks! |
Description
Fixes: #3541
Screenshots (optional)
Type of change
not work as expected)
Checklist:
./build/all.py
and the build passes./build/test.py
and all tests pass