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
Address AD review comments from Ben Kaduk #232
Conversation
All but two comments are addressed in this PR Signed-off-by: Dave Thaler <dthaler@microsoft.com>
@kaduk addressing most of your comments in this PR |
draft-ietf-teep-architecture.md
Outdated
@@ -1327,7 +1339,11 @@ from being disclosed to the TAM that distributes them. In such a scenario, | |||
the files can be encrypted end-to-end between a Trusted Component Signer and a TEE. However, there | |||
must be some means of provisioning the decryption key into the TEE and/or some | |||
means of the Trusted Component Signer securely learning a public key of the TEE that it can use to | |||
encrypt. One way to do this is for the Trusted Component Signer to run its own TAM so that it can | |||
encrypt. The Trusted Component Signer cannot necessarily even trust the |
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.
Drop a space here: "encrypt. The..."
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.
Done
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.
Thanks for the changes; look good.
As Ben stated in his review, some comments are better addressed in a different doc, so these are applied here in the arch doc. Signed-off-by: Dave Thaler <dthaler@microsoft.com>
Signed-off-by: Dave Thaler <dthaler@microsoft.com>
Merging per Ming's approval |
All but two AD comments are addressed in this PR
Signed-off-by: Dave Thaler dthaler@microsoft.com