-
Notifications
You must be signed in to change notification settings - Fork 13
feat: add flashblocks support to bera-reth #202
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
base: rezbera/block-assembler-fix
Are you sure you want to change the base?
feat: add flashblocks support to bera-reth #202
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
leverages the work in paradigmxyz/reth#20122 |
| /// .with_flashblocks_listeners(listeners); | ||
| /// let add_ons = BerachainAddOns::new(eth_api_builder); | ||
| /// ``` | ||
| pub fn new(eth_api_builder: BerachainEthApiBuilder) -> Self { |
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.
todo: builder pattern?
correctly consume flashblocks from base correctly consume flashblocks from base Update mod.rs Update api.rs Update api.rs progress started
ca1a914 to
88fe631
Compare
aeb681a to
86b62ae
Compare
This PR depends on paradigmxyz/reth#20122 and as such, updates the reth dependency to the latest reth branch which the PR #20122 targets.
Simply put, this PR spins up the Flashblocks listeners much in the same was as Optimism does in their RPC layer, then leverages RPC level overrides for implementations such as
transaction_receiptto retrieve and serve preconfirmed state