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.

Sign up for GitHub

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

Jump to bottom

clarify and expand TEE manufacturer assets #26

Merged
merged 2 commits into from
Apr 10, 2024
Merged

clarify and expand TEE manufacturer assets #26

merged 2 commits into from
Apr 10, 2024

Conversation

eriktaubeneck
Copy link
Collaborator

Small PR to address @martinthomson comment here:

It is quite possible that operational practices at the TEE manufacturer is good enough that they don't have access to the private keys, but they will have access to the "root" key.

We should assume that they have both.

Originally posted by @martinthomson in #14 (comment)

@eriktaubeneck eriktaubeneck mentioned this pull request Oct 21, 2022
Merged
martinthomson
martinthomson reviewed Oct 23, 2022
View reviewed changes
Copy link
Collaborator

@martinthomson martinthomson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

threat-model/readme.md Outdated Show resolved Hide resolved
@eriktaubeneck @martinthomson
Update threat-model/readme.md
d1ce4ce 
martinthomson's suggestions.

Co-authored-by: Martin Thomson <mt@lowentropy.net>
@AramZS AramZS added the call-for-consensus We are calling for participants to reach consensus. 2 weeks from being added or handled via agenda+ label Sep 11, 2023
@AramZS AramZS added Threat Model Document This pertains to the Threat Model document and is in an editorial mode. Merge Pending Change is set to be merged pending editor or chair action. Last chance to comment. and removed call-for-consensus We are calling for participants to reach consensus. 2 weeks from being added or handled via agenda+ labels Oct 30, 2023
@AramZS AramZS mentioned this pull request Apr 8, 2024
Closed
@AramZS AramZS merged commit 69ce412 into main Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martinthomson martinthomson martinthomson left review comments

Assignees
No one assigned
Labels
Merge Pending Change is set to be merged pending editor or chair action. Last chance to comment. Threat Model Document This pertains to the Threat Model document and is in an editorial mode.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@eriktaubeneck @martinthomson @AramZS