-
Notifications
You must be signed in to change notification settings - Fork 225
Remove UVM endorsement descriptor hardcoding at Start and Recovery #6869
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
Remove UVM endorsement descriptor hardcoding at Start and Recovery #6869
Conversation
…_at_start_and_recovery
…nd_recovery' of https://github.com/achamayou/CCF into remove_uvm_endorsement_descriptor_hardcoding_at_start_and_recovery
…_at_start_and_recovery
…_at_start_and_recovery
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.
To double check this PR moves the check of UVM endorsements from the startup path to the open decision by the consortium members.
So what root of trust is added to the kv, is this part of the open decision?
@cjen1-msft yes, that's exactly right. The did/feed/svn of the UVM endorsements validated at startup is captured in The recovery initial tx gets to set this again, on the basis of endorsements found on recovery node startup. |
I think this is all the change we need, because the UVM endorsement descriptor is already passed to
/create
and/create
writes the descriptor to the ledger/KV.