Skip to content

Better timeout handling of beacon chain data - #2960

Merged
sparrowDom merged 1 commit into
masterfrom
nicka/talos-timeouts
Aug 4, 2026
Merged

Better timeout handling of beacon chain data#2960
sparrowDom merged 1 commit into
masterfrom
nicka/talos-timeouts

Conversation

@naddison36

@naddison36 naddison36 commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Extend the beacon-state fetch timeout from 5 to 15 minutes.
  • Keep the timeout active while downloading the complete SSZ response body.
  • Add progress logs when response headers arrive and when the body download completes.
  • Log when a beacon-state request is aborted after reaching the timeout.

Background

Beacon-state responses are large and can take approximately seven minutes to download. The existing five-minute timeout was cleared as soon as the response headers arrived, leaving the subsequent response.arrayBuffer() operation without any timeout.

This caused Talos actions to remain in the running state indefinitely if the response body stalled.

The updated timeout covers both the initial request and the full body download while allowing enough time for the observed response duration.

This applies to all actions using the shared beacon-state loader, including daily_verify_deposits and daily_verify_balances.

Testing

  • Ran pnpm prettier:js.
  • Verified utils/beacon.js loads successfully.
  • Ran git diff --check.

@sparrowDom sparrowDom 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

@sparrowDom
sparrowDom merged commit 94c7c56 into master Aug 4, 2026
9 of 11 checks passed
@sparrowDom
sparrowDom deleted the nicka/talos-timeouts branch August 4, 2026 13:17
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