Skip to content

fix: retry failed HTTP calls and reduce check batch size#4

Merged
Agilulfo1820 merged 1 commit intovechain:mainfrom
reheatvet:fix/resilient-http-calls
Mar 11, 2026
Merged

fix: retry failed HTTP calls and reduce check batch size#4
Agilulfo1820 merged 1 commit intovechain:mainfrom
reheatvet:fix/resilient-http-calls

Conversation

@reheatvet
Copy link
Contributor

What's wrong

On my host, the SDK's HttpClient drops requests when too many are fired at once. One failed call crashes the whole cycle because
Promise.all rejects on any single failure. The node gets stuck in a crash-retry loop every 5 minutes.

What this does

  • Adds retry (3 attempts with backoff) to the base function so transient network failures don't kill the cycle. Reverts aren't retried.
  • Drops CHECK_BATCH from 50 to 10 so fewer calls are made at once.

Results

Tested on my host that previously crashed every cycle:

  • Before: crash loop, never completes
  • Retry only: works but takes ~4.5 min
  • Retry + smaller batches: ~28s per phase, no issues

Hope this is fine, let me know if you'd like any changes

@Agilulfo1820 Agilulfo1820 merged commit 28cce09 into vechain:main Mar 11, 2026
1 check passed
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