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

Offline storage demo not working #1768

Closed
danmo opened this issue Jan 12, 2019 · 7 comments
Closed

Offline storage demo not working #1768

danmo opened this issue Jan 12, 2019 · 7 comments
Labels
status: archived Archived and locked; will not be updated type: external An issue with an external dependency; not our issue; sometimes kept open for tracking

Comments

@danmo
Copy link

danmo commented Jan 12, 2019

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

What version of Shaka Player are you using?
Latest.

Can you reproduce the issue with our latest release version?
Yes.

Can you reproduce the issue with the latest code from master?

Yes.
Are you using the demo app or your own custom app?

Demo app.
If custom app, can you reproduce the issue using our demo app?

What browser and OS are you using?
Chrome on Mac

For embedded devices (smart TVs, etc.), what model and firmware version are you using?

What are the manifest and license server URIs?

What did you do?
image
When using the online shaka demo https://shaka-player-demo.appspot.com/demo/#asset=https://content.uplynk.com/847859273a4b4a81959d8fea181672a4.mpd?pr.version=2&pr.playenabler=B621D91F-EDCC-4035-8D4B-DC71760D43E9&pr.securitylevel=150;lang=en-US;build=uncompiled and trying to store/download a DRM encrypted asset I get above error.

I also have a production application that has not changed in months and now is failing with the same error, on DRM license request, seems the license server is rejecting the license request.

So this is happening in two unrelated applications using unrelated license servers etc.

What did you expect to happen?
be able to store drm asset.

What actually happened?
crashed with 6007 1001 errors.

@theodab
Copy link
Collaborator

theodab commented Jan 14, 2019

All of the Verizon Digital Media Services assets seem to be un-storable. Odd. I'll look into it.

@danmo
Copy link
Author

danmo commented Jan 14, 2019

Getting same error with an Azure Media Services license server on a production application. It fails exactly like the demo but AMS doesn't provide any error message besides the 400 bad request response. Non-persistent license requests are successful.

@joeyparrish
Copy link
Member

@TheJohnBowers, is it intentional that requests for persistent licenses are denied by the VDMS license server?

Since license servers generally are allowed to deny requests for persistent licenses, I think we may need to flag which of our encrypted demo assets permit storage offline. The ones we (Shaka) control all permit storage, but we can't control the policy settings of third parties.

@TheJohnBowers
Copy link
Contributor

We have a default widevine policy for our license server. This policy is to issue non persistent licenses, so this is not unexpected. Our license server can issue persistent licenses, but we do not do that by default. For the purposes of the demo player we can issue a persistent license though. That would probably avoid people re-submitting this bug over and over. If we simply add rmp.policy_overrides.can_persist=True to the mpd url as a parameter, then our license server should issue a persistent license and storing will then be possible.

@joeyparrish
Copy link
Member

I don't think we care if these particular assets can get a persistent license or not, so feel free to change it or not as you see fit, @TheJohnBowers. But I think this highlights the need to flag certain third-party assets as not storable, so that the demo app doesn't offer a capability it can't provide.

@theodab theodab added type: external An issue with an external dependency; not our issue; sometimes kept open for tracking and removed needs triage labels Jan 15, 2019
shaka-bot pushed a commit that referenced this issue Jan 15, 2019
Not all of our assets support offline storage. For example, livestreams
inherently cannot be stored. In addition, some DRM assets do not have
the right policy settings to be stored offline.
This change marks some assets as being offline-compatible, and disables
the button if the asset is not.
It also removes some obsolete code that somehow ended up getting into
the asset structure.

Issue #1768

Change-Id: I22671bd82e9c5e4645158a6e0a2f8f91893082c7
@theodab
Copy link
Collaborator

theodab commented Jan 15, 2019

Ok, this CL will mark which assets are storable online and which aren't. It's just a field in the asset structure, so it won't be able to tell for custom assets, but it should prevent more misunderstandings in the future.

@theodab theodab closed this as completed Jan 15, 2019
joeyparrish pushed a commit that referenced this issue Jan 19, 2019
Not all of our assets support offline storage. For example, livestreams
inherently cannot be stored. In addition, some DRM assets do not have
the right policy settings to be stored offline.
This change marks some assets as being offline-compatible, and disables
the button if the asset is not.
It also removes some obsolete code that somehow ended up getting into
the asset structure.

Issue #1768

Backported to v2.4.x

Change-Id: I22671bd82e9c5e4645158a6e0a2f8f91893082c7
@joeyparrish
Copy link
Member

Offline storage restriction in demo app cherry-picked for v2.4.6.

@shaka-project shaka-project locked and limited conversation to collaborators Mar 16, 2019
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Apr 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated type: external An issue with an external dependency; not our issue; sometimes kept open for tracking
Projects
None yet
Development

No branches or pull requests

5 participants