Skip to content

feat: support block number in required-block-hashes parameter#18546

Merged
mattsse merged 5 commits intoparadigmxyz:mainfrom
leniram159:block-number
Nov 12, 2025
Merged

feat: support block number in required-block-hashes parameter#18546
mattsse merged 5 commits intoparadigmxyz:mainfrom
leniram159:block-number

Conversation

@leniram159
Copy link
Contributor

@leniram159 leniram159 commented Sep 18, 2025

Implements support for block number in the --required-block-hashes parameter as requested in issue #18543.

  • Accept BlockNumHash instead of B256 for required block hashes
  • Add parser for block_number=hash format (e.g., 23115201=0x1234...)
  • Skip block checks when peer's latest block is below required block number
  • Maintain backward compatibility with hash-only format

@leniram159 leniram159 requested a review from gakonst as a code owner September 18, 2025 11:09
Copy link
Contributor

@Peponks9 Peponks9 left a comment

Choose a reason for hiding this comment

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

hey, try to add Closes #number_of_issue to the description so this pr will be linked to the issue. might tell the maintainers to check soon :)

@leniram159
Copy link
Contributor Author

hey, try to add Closes #number_of_issue to the description so this pr will be linked to the issue. might tell the maintainers to check soon :)

Thanks for the advice!!!!!!

@leniram159
Copy link
Contributor Author

Closes #18543 cc @mattsse

@github-actions github-actions bot added the S-stale This issue/PR is stale and will close with no further activity label Oct 17, 2025
@github-actions github-actions bot closed this Oct 26, 2025
@github-project-automation github-project-automation bot moved this from Backlog to Done in Reth Tracker Oct 26, 2025
@leniram159
Copy link
Contributor Author

@mattsse Hi boss, closed by bots, would you please re-open this?

@mattsse
Copy link
Collaborator

mattsse commented Oct 29, 2025

sorry @leniram159

forgot to review this in time, we still want this, thanks for the ping

@mattsse mattsse reopened this Oct 29, 2025
@github-project-automation github-project-automation bot moved this from Done to In Progress in Reth Tracker Oct 29, 2025
@github-actions github-actions bot removed the S-stale This issue/PR is stale and will close with no further activity label Oct 30, 2025
Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

lgtm

sorry forgot about this one

@mattsse mattsse enabled auto-merge November 12, 2025 13:37
@mattsse mattsse added this pull request to the merge queue Nov 12, 2025
Merged via the queue into paradigmxyz:main with commit 6c1296d Nov 12, 2025
42 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Nov 12, 2025
emhane pushed a commit to op-rs/op-reth that referenced this pull request Nov 24, 2025
…gmxyz#18546)

Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants