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

"tracked" sessions: "record of key usage" is undefined #81

Closed
ddorwin opened this issue Aug 27, 2015 · 3 comments
Closed

"tracked" sessions: "record of key usage" is undefined #81

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

Comments

@ddorwin
Copy link
Contributor

ddorwin commented Aug 27, 2015

record of key usage, which was added by pull request #54, is used as a variable in several algorithms but not clearly defined anywhere. It should probably be "this object's record of key usage" and be initialized in createSession().

In addition, it would probably be helpful to provide information about the variable. For example, that the record contains entries, as referenced in section 7.5.2 Encrypted Block Encountered. It might also help to make it plural somehow, since it has records for all keys in the session. (Both of these will be moot if the change proposed by #80 is made.)

first decrypt time and latest decrypt time are also undefined in section 7.5.2

@mwatson2
Copy link
Contributor

mwatson2 commented Sep 1, 2015

The record of key usage is defined in the description of the "tracked" session type in Section 5. I agree we should explicitly initialize it.

@mwatson2
Copy link
Contributor

mwatson2 commented Sep 3, 2015

I believe this is addressed by the changes proposed in #87

@ddorwin ddorwin modified the milestone: V1 Oct 20, 2015
@mwatson2
Copy link
Contributor

mwatson2 commented Nov 5, 2015

Addressed by PR #87.

@mwatson2 mwatson2 closed this as completed Nov 5, 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

2 participants