search

paradigmxyz / reth

Modular, contributor-friendly and blazing-fast implementation of the Ethereum protocol, in Rust
https://reth.rs/
Apache License 2.0
3.58k stars 933 forks source link

Historical EIP-1186 `eth_getProof` support for op-stack Withdrawal proofs #8147

Open 0xOsiris opened 2 months ago

0xOsiris commented 2 months ago

Describe the feature

Currently Reth does not offer support for eth_getProof on historical blocks. This breaks compatibility with the op-stack having Reth running as the sequencing execution client as Reth will not be able to generate withdrawal proofs for the proposer to post to L1 (except at the tip).

Additional context

Ref: https://github.com/ethereum-optimism/optimism/blob/89a3a5f32b79b2447690cdac70b1df498ffdb25c/op-node/withdrawals/utils.go#L102

Related to: https://github.com/paradigmxyz/reth/issues/5246

0xKitsune commented 2 months ago

Just a quick update, @0xOsiris and I have started working on this. I will open a draft PR with a few questions a little later today.

github-actions[bot] commented 1 month ago

This issue is stale because it has been open for 21 days with no activity.

github-actions[bot] commented 1 month ago

This issue was closed because it has been inactive for 7 days since being marked as stale.

nikkolasg commented 1 month ago

Hey - any chance this issue might be worked on at some point ? I see it dropped now but it would be a fantastic and important feature to have for reth !

onbjerg commented 1 month ago

Thanks for flagging, reopened it for consideration:)

jaeaster commented 3 weeks ago

+1 for this. We decided to go with Besu until this is ready

denis-pingin commented 3 weeks ago

This is a much needed feature to be able to use Reth for OP stack rollups