Skip to content

Bump the Firehose forks to world-chain-v2.4.3-fh3.1-2 - #9

Merged
sduchesneau merged 1 commit into
release/2.xfrom
bump/reth-keccak-cap-256
Sep 17, 2026
Merged

sduchesneau merged 1 commit into
release/2.xfrom
bump/reth-keccak-cap-256

Conversation

@sduchesneau

@sduchesneau sduchesneau commented Sep 17, 2026

Copy link
Copy Markdown

Bumps both Firehose forks from world-chain-v2.4.3-fh3.1-1 to world-chain-v2.4.3-fh3.1-2 and opens the v2.4.3-fh3.1-2 changelog section. Both tags now exist.

The only change is the 256-byte cap on keccak preimages. Values in Call.keccak_preimages could reach 65536 bytes each (131072 hex characters). The map is there so a consumer can walk a storage slot back to the expression that produced it, and Solidity's slot derivations are small: 32 bytes for a dynamic array or a bytes/string longer than 31 bytes, 64 bytes for a mapping with a value-type key, and 32 bytes plus the key for a mapping(string => V). 256 bytes covers all of those with 224 bytes of room for a dynamic key. Oversized preimages are dropped rather than truncated, since a truncated preimage no longer hashes back to its map key.

No source change is needed here — the cap lives in the reth fork's crates/firehose/src/inspector.rs.

Cargo.toml and Cargo.lock were updated by hand, substituting the tag string and the locked revs across 148 Cargo.toml and 148 Cargo.lock entries:

Fork Pins rev
streamingfast/reth 104 0895359337b85f5464cf (streamingfast/reth#43)
streamingfast/optimism 44 49b6568664497043385e (streamingfast/optimism#21)

Neither commit touches a Cargo.toml, so the dependency graph is unchanged and nothing else in the lock moves.

@dfuse-bot
dfuse-bot requested a review from billettc September 17, 2026 20:28
Brings the 256-byte cap on keccak preimages.
@sduchesneau
sduchesneau force-pushed the bump/reth-keccak-cap-256 branch from 6a8884b to 31d9f4d Compare September 17, 2026 20:38
@sduchesneau
sduchesneau merged commit 649d26b into release/2.x Sep 17, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant