fix mutual-remote-attestatoin #149
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…st_key_provisioning
…tches calculates the state from 0.
…a state while provisioning secrets. And put it at the very beginning of `fn worker`
clangenb
force-pushed
the
cl-fix-running-multiple-workers
branch
from
513187f to
316fafd
Compare
brenzi
pushed a commit
to encointer/encointer-worker
that referenced
this pull request
Sep 8, 2020
* [worker] initialize chain after keys have been provisioned * [worker] fix cli for key_provisioning tests * [worker/mu-ra] pass shard that should be exchanged is passed to request_key_provisioning * [enclave/cert] fix mr_enclave fetch and comparison * [enclave/tls_ra] remove state exchange from ra as every new worker fetches calculates the state from 0. * [worker] change get_most_recent_worker -> get_first_worker that is not equal to self * [worker] ensure_shard_initialized now in any case as we do not fetch a state while provisioning secrets. And put it at the very beginning of `fn worker` * [enclave/tls_ra] remove unsafe from request_key_provisioning * bump version: 0.6.5-sub2.0.0-alpha.7
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request fixes the mutual remove attestation, which was broken with to the introduction of shards. The enclave state is no longer provisioned in mutual remote attestation as each enclave can reproduce past worker requests upon block inclusion from the chain relay.
mutual remote attestation
worker
get_first_worker_not_equal_to_selfinstead ofget_worker_for_shardto request key provisioning as no state is exchanged anymore in mura. Closes w2 does not requests keys from w1 if w1 has not yet published a state update #146