Repository navigation
Trace the Keystore account changes - #37
Merged
Merged
Conversation
Records the EIP-8130 create and config change entries instead of stopping the node, and the sender's authentication when it is not a secp256k1 signature.
sduchesneau
added this pull request to stack #38
October 20, 2026 20:09
sduchesneau
marked this pull request as ready for review
October 20, 2026 20:14
# Conflicts: # Cargo.lock
# Conflicts: # Cargo.lock # Cargo.toml # crates/execution/firehose-tests/tests/eip8130.rs # crates/execution/firehose/src/extras.rs
maoueh
approved these changes
Oct 21, 2026
# Conflicts: # crates/execution/firehose-tests/Cargo.toml # crates/execution/firehose-tests/tests/eip8130.rs
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
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.
Stacked on #35. Traces the Keystore account changes of EIP-8130 (enabled by Zenith) instead of stopping the node:
OpPreTxAdjustmaps create entries (user salt, code, initial actors) and config change entries (channel, sequence, changes with their ABI payloads, admin auth) intoaccount_changes, per Add the Keystore account changes to the account abstraction model firehose-ethereum#264.authenticator || datainsender_auth, and emptyv/r/s. Before, the last 65 bytes of that blob were read as a secp256k1 signature.AccountCreated/ActorAuthorizedlogs are journal writes recorded on the root call by the existing path.sender_authis recorded as signed.The dependencies are the same as #35:
firehose-tracer5.7.0 (which already has the Keystore bindings, streamingfast/evm-firehose-tracer-rs#40) and the reth tagbase-v2.5.2.8-fh3.1-2.