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

Implement checksum checking #572

Closed
Tracked by #322
sideninja opened this issue Sep 25, 2024 · 1 comment
Closed
Tracked by #322

Implement checksum checking #572

sideninja opened this issue Sep 25, 2024 · 1 comment

Comments

@sideninja
Copy link
Contributor

sideninja commented Sep 25, 2024 •
edited
Loading

The local register index should validate the changes to the state by using the checksum provided by the EN. That is dependent on the PR onflow/flow-go#6456

After each transaction we can produce the checksum and compare that checksum with the checksum we receive from the EN via the transaction executed event. If those checksum match we can be sure the state is correct.
@sideninja sideninja mentioned this issue Sep 25, 2024
Closed
@j1010001
Copy link
Member

Closing, duplicate of onflow/flow-go#6539, implemented in onflow/flow-go#6544, onflow/flow-go#6545, onflow/flow-go#6546

@github-project-automation github-project-automation bot moved this to ✅ Done in 🌊 Flow 4D Oct 16, 2024
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
🌊 Flow 4D
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@j1010001 @sideninja