Skip to content

"tracked" sessions: record of usage should track session usage, not individual key usage #80

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

Closed
ddorwin opened this issue Aug 27, 2015 · 5 comments
Assignees
Milestone

Comments

@ddorwin
Copy link
Contributor

ddorwin commented Aug 27, 2015

The text from pull request #54 says "the record of key usage" contains an "entry for block key ID." This means that there is a record of usage for each key in the session. This granularity seems unnecessary and has costs, including:

  1. Complicating the CDM, license protocol, and license server implementation
  2. Requiring the author's server to aggregate the intervals from all keys to come up with the total playback period for the session

The spec generally treats sessions as a single entity, including closing, removing, and expiration time, and it's unclear why this should be different. If an author really needs different usage data, he or she can use multiple sessions.

@mwatson2
Copy link
Contributor

mwatson2 commented Sep 1, 2015

Agreed. We should record the overall start and end times and the ids of the keys that were available.

@jdsmith3000
Copy link
Contributor

Tracking by session rather than individual keys aligns with Microsoft's thinking. We support this recommendation.

@paulbrucecotton
Copy link

If we have initial agreement here, can one of editors volunteer to prepare a pull request for this issue?

@mwatson2
Copy link
Contributor

mwatson2 commented Sep 3, 2015

Yes, I will do that. Also for the others where there is agreement or at least
a direction to discuss in more detail.

mwatson2 added a commit to mwatson2/encrypted-media that referenced this issue Sep 3, 2015
@mwatson2
Copy link
Contributor

mwatson2 commented Sep 3, 2015

PR: #87

mwatson2 added a commit to mwatson2/encrypted-media that referenced this issue Oct 19, 2015
@ddorwin ddorwin modified the milestone: V1 Oct 20, 2015
mwatson2 added a commit to mwatson2/encrypted-media that referenced this issue Oct 27, 2015
mwatson2 added a commit to mwatson2/encrypted-media that referenced this issue Oct 28, 2015
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

No branches or pull requests

4 participants