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

encrypted state must contain the hash of last request #36

Closed
brenzi opened this issue Jul 2, 2019 · 4 comments
Closed

encrypted state must contain the hash of last request #36

brenzi opened this issue Jul 2, 2019 · 4 comments
Milestone
M7

Comments

@brenzi
Copy link
Collaborator

brenzi commented Jul 2, 2019

In order for a newly joining worker to know where to catch up after synching with a previous worker (#35), we should write the hash of the request that caused this state update to the state itself. This way, the joining worker can scan past events for this request and catch up.
@brenzi brenzi mentioned this issue Jul 2, 2019
Closed
@brenzi
Copy link
Collaborator Author

brenzi commented Sep 27, 2019

this is actually missing in #61

@brenzi
Copy link
Collaborator Author

brenzi commented Jan 30, 2020

this should go hand in hand with introducing versioned state. Intoduce an enum type that specifies the state version like for substrate metadata. The versioned type then includes

  • raw state
  • short journal of processed requests (1-2 blocks worth of requests?)
  • latest block hash
  • worker enclave signature

@brenzi brenzi added this to the M milestone Jan 30, 2020
@brenzi
Copy link
Collaborator Author

brenzi commented Feb 6, 2020

connected to #38

@brenzi
Copy link
Collaborator Author

brenzi commented May 25, 2020

since a newly joining worker can now re-sync from ChainRelay directly, this is not necessary anymore

@brenzi brenzi closed this as completed May 25, 2020
brenzi pushed a commit that referenced this issue Dec 12, 2020
@clangenb
Extend trusted getters (#39)
2850674 
* Implement `TrustedGetter::meetup_registry(AccountId, CurrencyIdentifier)` Fixes #34 
* Reduce `TrustedGetter::meetup_index_and_location` to `TrustedGetter::meetup_index`. Fixes #35 
* Rename `TrustedGetter::registration` -> `TrustedGetter::participant_index`. Fixes #36 

Successfully run botscript.

The new `TrustedGetter::meetup_registry` has only been tested weakly from EncointerWallet js side, such that should return an empty registry.
echevrier pushed a commit that referenced this issue Aug 4, 2022
@clangenb
Merge pull request #36 from integritee-network/ec_runtime_from_metadata
a07c576 
Parentchain's metadata in enclave
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
M7
Development

No branches or pull requests
1 participant
@brenzi