Skip to content

fix: gate Gloas/Heze event topics by fork activation#638

Merged
barnabasbusa merged 3 commits intomasterfrom
fix/eventstream-fork-gating
Apr 14, 2026
Merged

fix: gate Gloas/Heze event topics by fork activation#638
barnabasbusa merged 3 commits intomasterfrom
fix/eventstream-fork-gating

Conversation

@barnabasbusa
Copy link
Copy Markdown
Collaborator

@barnabasbusa barnabasbusa commented Apr 14, 2026

Dora subscribes to execution_payload_available, execution_payload_bid and inclusion_list unconditionally. Clients that don't implement inclusion_list yet (lodestar v1.40, pre-Heze prysm) 400 the whole SSE bundle with Invalid topic, so dora loses all three advanced streams and spams a warning every ~10s.

Include Gloas topics only when Gloas (EIP-7732) is active, and inclusion_list only when Heze (EIP-7805) is active. Return from runClientLogic when the mask changes so the outer retry loop reconnects with the new topics.

🤖 Generated with Claude Code

Subscribing to inclusion_list on clients that don't implement it yet
(lodestar v1.40, pre-Heze prysm) 400s the whole SSE bundle, taking
execution_payload_available / _bid down with it. Include per-fork topics
only when the fork is active; reconnect on activation.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@barnabasbusa barnabasbusa force-pushed the fix/eventstream-fork-gating branch from a0914b8 to a4f81d4 Compare April 14, 2026 09:12
@barnabasbusa barnabasbusa enabled auto-merge April 14, 2026 09:13
@barnabasbusa barnabasbusa force-pushed the fix/eventstream-fork-gating branch from c00bcd6 to a5b9ee2 Compare April 14, 2026 09:26
…basic stream

Previously the mask-change check returned an error to force the outer
retry loop to reconnect everything. That dropped head/block events for
~10s at the fork boundary. Instead, split the advanced stream into
per-fork ancillaries and expose UpdateEvents on BeaconStream so
clientlogic spawns a new SSE request for just the newly-activated
topics, leaving basic/other streams untouched.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@barnabasbusa barnabasbusa merged commit 45319f2 into master Apr 14, 2026
4 checks passed
@barnabasbusa barnabasbusa deleted the fix/eventstream-fork-gating branch April 14, 2026 09:37
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