Skip to content

Retry the gitleaks download - #140

Merged
louisbels merged 1 commit into
mainfrom
ci/harden-gitleaks-download
Jul 26, 2026
Merged

Retry the gitleaks download#140
louisbels merged 1 commit into
mainfrom
ci/harden-gitleaks-download

Conversation

@louisbels

Copy link
Copy Markdown
Member

The secrets job failed on #139 with:

curl: (35) Recv failure: Connection reset by peer
##[error]Process completed with exit code 35.

The gitleaks binary never downloaded, so nothing was scanned. A re-run passed untouched.

Two reasons this is worth fixing rather than re-running when it happens:

  • Since Harden the app: data-loss bugs, real CI gates, security and UX fixes #133 this job gates the deploy. A blip in GitHub Releases now blocks a release outright.
  • The failure looks exactly like a detected secret — same red check, same "secrets failed" email — until you open the log and read the curl error. That is a bad few minutes for whoever is on the receiving end.

Five retries with a connect timeout make the transient case self-healing; a genuine outage still fails the job.

Verified with actionlint.

The secrets job failed on a PR with `curl: (35) Recv failure: Connection reset
by peer` — the binary download never completed, so nothing was ever scanned.

Since the deploy gate now requires this job, a blip in GitHub Releases blocks a
release outright, and the failure reads like a detected secret until you open
the log. Five retries with a connect timeout make the transient case
self-healing while a genuine outage still fails.
@louisbels
louisbels merged commit 4a36544 into main Jul 26, 2026
12 checks passed
@louisbels
louisbels deleted the ci/harden-gitleaks-download branch July 26, 2026 15:47
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.

1 participant