Skip to content

fix: update reth to add block payload size validation #221

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

Merged
merged 9 commits into from
Jul 28, 2025

Conversation

yiweichi
Copy link
Member

This PR is for Issue: scroll-tech/reth#281

Related PR: scroll-tech/reth#287
The Reth repo PR adds a new limit block_da_size_limit when building a new block. Make sure we won't build a block that's too large to be able to put into a Ethereum blob.

@yiweichi yiweichi changed the title fix: set ScrollPayloadBuilderBuilder fix: update reth to add block payload size validation Jul 27, 2025
@yiweichi yiweichi requested review from frisitano and greged93 and removed request for frisitano July 27, 2025 17:38
Copy link
Collaborator

@frisitano frisitano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, left one comment inline.

frisitano
frisitano previously approved these changes Jul 28, 2025
Copy link
Collaborator

@frisitano frisitano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merge reth branch feat-add-max-da-block-size, repoint dep and lgtm

Copy link
Collaborator

@frisitano frisitano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@yiweichi yiweichi merged commit 3dce7a7 into main Jul 28, 2025
12 checks passed
@yiweichi yiweichi deleted the feat-add-max-da-block-size branch July 28, 2025 14:29
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.

2 participants