Skip to content

fix: retry transient artifact download failures - #256

Merged
hetaoBackend merged 1 commit into
mainfrom
fix/artifact-download-retries
Sep 20, 2026
Merged

hetaoBackend merged 1 commit into
mainfrom
fix/artifact-download-retries

Conversation

@SaladDay

@SaladDay SaladDay commented Sep 20, 2026 •

Copy link
Copy Markdown
Collaborator

Why

This CI build failed while downloading the mcode-tools archive from npm: the TLS connection was reset (ECONNRESET). The download had no retry, so one temporary network failure stopped the build.

Changes

  • Retry temporary connection failures, interrupted downloads, timeouts and HTTP 408/429/500/502/503/504 up to three times, with 1-second and 2-second backoff. Respect Retry-After up to 30 seconds.
  • Keep the 120-second deadline per attempt and log retries with the final failure cause. Permanent errors and integrity failures still fail immediately; cache and integrity checks are preserved.
  • Add regression tests for recovery, retry limits, permanent errors and corrupt archives.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@hetaoBackend
hetaoBackend merged commit a3ae605 into main Sep 20, 2026
13 checks passed
@hetaoBackend
hetaoBackend deleted the fix/artifact-download-retries branch September 20, 2026 14:26
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