Skip to content

Make wordings clearer for self build payload signature verification#5002

Merged
jtraglia merged 4 commits intoethereum:masterfrom
ensi321:clarify-execution-payload-gossip-self-build
Mar 13, 2026
Merged

Make wordings clearer for self build payload signature verification#5002
jtraglia merged 4 commits intoethereum:masterfrom
ensi321:clarify-execution-payload-gossip-self-build

Conversation

@ensi321
Copy link
Contributor

@ensi321 ensi321 commented Mar 13, 2026

Currently wordings for signature verification for p2p execution_payload makes sense for non self-build scenario but is confusing for self-build scenario.

Update wordings such that the rule works for both.

ensi321 and others added 2 commits March 9, 2026 18:36
…payloads

The gossip validation rule for `signed_execution_payload_envelope.signature`
only mentioned verifying against the builder's public key. For self-built
payloads (`BUILDER_INDEX_SELF_BUILD`), the signature is verified against the
proposer's public key instead. Clarify this and reference
`verify_execution_payload_envelope_signature`.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the gloas label Mar 13, 2026
Copy link
Member

@jtraglia jtraglia left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! I'll schedule this to auto-merge after the CI checks pass. Just needs make lint.

@jtraglia jtraglia enabled auto-merge (squash) March 13, 2026 00:39
auto-merge was automatically disabled March 13, 2026 07:10

Head branch was pushed to by a user without write access

@jtraglia jtraglia merged commit 85ab2d2 into ethereum:master Mar 13, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants