-
Notifications
You must be signed in to change notification settings - Fork 7
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
Proposal to add multiplicity of Measurement Maps #265
Conversation
Signed-off-by: Yogesh Deshpande <yogesh.deshpande@arm.com>
I am working on scanning the conditionality checks, so will make more commits to the PR! |
This is a slight modification to already closed PR #181 in which the new PR introduces |
@deeglaze If you feel al-right, feel free to make more commits on the branch! |
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.
What is the meaning of this tag? I'm not sure why you need one when arrays and maps have different CBOR major type tags.
I don't have push rights to this branch, so I have suggestions in my own commit on top in multiplicity...deeglaze:draft-ietf-rats-corim:multiplicity |
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.
A straightforward solution is added to the tcg-compatible branch https://github.com/ietf-rats-wg/draft-ietf-rats-corim/tree/tcg-compatible
It includes examples that are backward compatible with the TCG specs. It doesn't seem to need a CBOR tag.
Agreed it doesn't need the tag, but I do believe that conditional endorsements need to be updated to describe multiple mkeys at a time. |
Conditional endorsements were an IETF invention. Hence, no backwards compatibility concerns. The conditional endorsement triple was already updated to add multiplicity. The conditional series triple has the needed multiplicity properties already (at least I haven't had any requests to change it from the people I work with). |
Contrast with #268 |
The work proposed under this PR will be undertaken by PR: #268 Hence closing this PR |
Modern day attesters have Multiple Measured Elements, within a Target Environment. This PR intends to allow multiple measured elements to be expressed using a single Reference Value Triple.