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.

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

ReceiptsRoot not computed correctly #8073

Closed
2 tasks done
NiloCK opened this issue Jun 5, 2024 · 0 comments · Fixed by #8085
Closed
2 tasks done

ReceiptsRoot not computed correctly #8073

NiloCK opened this issue Jun 5, 2024 · 0 comments · Fixed by #8085
Labels
T-bug Type: bug

Comments

@NiloCK
Copy link
Contributor

NiloCK commented Jun 5, 2024

Component

Anvil

Have you ensured that all of these are up to date?

  • Foundry
  • Foundryup

What version of Foundry are you on?

forge 0.2.0 (a66a0cf 2024-05-31T00:57:36.573315267Z)

What command(s) is the bug in?

anvil

Operating System

Linux

Describe the bug

I believe I'm hitting a discrepancy between geth and anvil in receipt root calculation:

ERROR[06-05|14:42:17.255] sequencer temporarily failed to start building new block err=
 "temp: failed to fetch L1 block info and receipts: failed to fetch list of receipts:
   expected receipt root 0x987c20331e3e9b054da27a8269e5e558c0db18344de1a8413d0eaec32d63884d but computed 0x6a3f473ade72ee4cb1d10a27d4cae4e2bbcdc3394e94de33c3f76654568ecf85 from retrieved receipts"

This issue resembles #7698, which was fixed in #7718.

See

let receipts_root = trie::ordered_trie_root(receipts.iter().map(alloy_rlp::encode));

As far as I can tell, this is the only place in anvil where receipts_root is calculated, and it uses the same rlp::encode function that was replaced in the above fix for the transaction root.

Drive-by request: Out of necessity, I'm currently using a build from the open PR #7918, which was approved with CI failures, but now seems stagnant. Can it be merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-bug Type: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant