w3c / encrypted-media

Encrypted Media Extensions
https://w3c.github.io/encrypted-media/
Other
180 stars 80 forks source link

Provide a definition for expiration time #425

Closed ddorwin closed 7 years ago

ddorwin commented 7 years ago

The term "expiration time" is used multiple times in the algorithms (e.g., "If cdm has changed the expiration time of session...") yet there is no definition for it.

The definition of the expiration attribute says:

The time after which the key(s) in the session will no longer be usable for decryption, or NaN if no such time exists or if the license explicitly never expires, as determined by the CDM.

While this provides a definition, the relationship to the instances in the algorithm is indirect, especially since those algorithms do not directly modify expiration.

We should define "Expiration Time" in the Definitions section and move the first part of the sentence there. We would then have:

Expiration Time The time after which key(s) will no longer be usable for decryption.

expiration "The expiration time for key(s) in the session, or NaN if no such time exists or if the license explicitly never expires, as determined by the CDM.

If cdm has changed the expiration time of session...

jdsmith3000 commented 7 years ago

This seems like a good idea, but perhaps is more than necessary. I believe execution time is commonly understood, and our goal has been to tighten up and use a better definition of time. There are some loose ends still there, but it would only take some format cleanup, removing the section from the time reference, and purging the last stray millisecond reference from the spec.

ddorwin commented 7 years ago

I created PR #426 for discussion. One advantage of this is that it addresses the "expressed as time" issue without new text in the algorithms.