Skip to content

Conversation

storopoli
Copy link
Member

@storopoli storopoli commented Sep 12, 2025

Description

Custom RLPx subprotocol to gossip the head block hash in alpen-reth and reth-node.

Blocked 🪨 by #1035 which has the PreConsensusTracker trait that this PR should use.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature/Enhancement (non-breaking change which adds functionality or enhances an existing one)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactor
  • New or updated tests
  • Dependency Update

Notes to Reviewers

Checklist

  • I have performed a self-review of my code.
  • I have commented my code where necessary.
  • I have updated the documentation if needed.
  • My changes do not introduce new warnings.
  • I have added (where necessary) tests that prove my changes are effective or that my feature works.
  • New and existing tests pass with my changes.
  • I have disclosed my use of AI
    in the body of this PR.

Related Issues

STR-1684

Copy link
Contributor

github-actions bot commented Sep 12, 2025

Commit: cbf9df3

SP1 Execution Results

program cycles success
Bitcoin Blockspace 70,870
EVM EE STF 1,172,673
CL STF 377,821
Checkpoint 4,229

@storopoli storopoli force-pushed the STR-1684-ee-p-2-p-broadcast-gossip-of-head-block-hash branch from c8d9caa to 0347523 Compare September 15, 2025 19:20
@storopoli
Copy link
Member Author

@sapinb please take a look at this.
See if this the approach that you were thinking with implementing a custom RLPx subprotocol to gossip the head block hash.

Maybe we should just gossip the FixedBytes<32> for the actual block hash instead of the very complex Header?

@storopoli storopoli force-pushed the STR-1684-ee-p-2-p-broadcast-gossip-of-head-block-hash branch from 0347523 to 7048ed3 Compare September 15, 2025 19:23
@storopoli storopoli self-assigned this Sep 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant