-
Notifications
You must be signed in to change notification settings - Fork 176
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
Verify Eth Headers #1010
base: main
Are you sure you want to change the base?
Verify Eth Headers #1010
Conversation
…de from global args
… bat/eth-full-node
…t syncinc worked correctly
…ully before starting anoma node
…ons are included on chain via a protocol tx
…eum/ethash) to verify ethereum headers.
…oring and feature flags for wasm compatibility
@@ -127,7 +127,10 @@ audit: | |||
test: test-unit test-e2e test-wasm | |||
|
|||
test-e2e: | |||
RUST_BACKTRACE=1 $(cargo) test e2e -- --test-threads=1 | |||
RUST_BACKTRACE=1 $(cargo) test e2e \ |
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.
is this change needed? It seems to work with the defaults
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.
I did that at first to use the mock eth node, but I forgot the we compile as part of the e2e test. So it is not needed.
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.
The code changes LGTM, we need to resolve the CI before merging (and changelog pls)
Co-authored-by: Tomas Zemanovic <tomas@heliax.dev>
…bat/verify-eth-headers
pls update wasm |
the design of the bridge has been revised and so this PR won't be used (remains open for salvaging common parts) |
This PR builds on #995
Adds an ethash algorithm (mostly cribbed from here) to verify ethereum headers. Because the hash must use a pseudorandom cache that is
we add a background process to precompute this cache, keeping verification times fast.
This required refactoring the Anoma types for interacting with Ethereum types.
N.B. This requires the latest commit of Tendermint ABCI++ branch