Skip to content

Emit SSE: execution_payload_gossip - #9063

Merged
mergify[bot] merged 1 commit into
sigp:unstablefrom
dknopik:sse-exec-payload-gossip
Mar 31, 2026
Merged

Emit SSE: execution_payload_gossip#9063
mergify[bot] merged 1 commit into
sigp:unstablefrom
dknopik:sse-exec-payload-gossip

Conversation

@dknopik

@dknopik dknopik commented Mar 31, 2026

Copy link
Copy Markdown
Member

Proposed Changes

Emit execution_payload_gossip on successful gossip verification of an execution payload. This is done as last step inside the verification function.

Additional information

This is following the established pattern for other SSEs, e.g.:

// Beacon API block_gossip events
if let Some(event_handler) = chain.event_handler.as_ref()
&& event_handler.has_block_gossip_subscribers()
{
event_handler.register(EventKind::BlockGossip(Box::new(BlockGossip {
slot: block.slot(),
block: block_root,
})));
}

Builds upon #9053.

@dknopik
dknopik requested a review from eserilev March 31, 2026 11:10
@dknopik dknopik added ready-for-review The code is ready for review HTTP-API gloas labels Mar 31, 2026

@eserilev eserilev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@eserilev eserilev added the ready-for-merge This PR is ready to merge. label Mar 31, 2026
@mergify mergify Bot added the queued label Mar 31, 2026
@mergify

mergify Bot commented Mar 31, 2026

Copy link
Copy Markdown

Merge Queue Status

This pull request spent 29 minutes 22 seconds in the queue, including 27 minutes 19 seconds running CI.

Required conditions to merge

mergify Bot added a commit that referenced this pull request Mar 31, 2026
@mergify
mergify Bot merged commit 037b263 into sigp:unstable Mar 31, 2026
38 checks passed
@mergify mergify Bot removed the queued label Mar 31, 2026
@dknopik
dknopik deleted the sse-exec-payload-gossip branch April 24, 2026 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gloas HTTP-API ready-for-merge This PR is ready to merge. ready-for-review The code is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants