Skip to content

Release: gitleaks download retry (develop -> main) - #247

Merged
mahmutkaya merged 1 commit into
mainfrom
develop
Sep 8, 2026
Merged

mahmutkaya merged 1 commit into
mainfrom
develop

Conversation

@mahmutkaya

Copy link
Copy Markdown
Contributor

Squash-merged since the last release:

#245)

A single curl made a REQUIRED check fail in 3s when GitHub's release CDN
returned a 504 (run 32473558686, release PR #176) — on a diff that could not
possibly contain a secret — and the only recovery was a human re-run.

Adds --retry 3 --retry-delay 2 --retry-all-errors to the install step's curl,
per the issue. --retry-all-errors is the load-bearing flag: under -f a clean
HTTP 504 response exits 22 without being retried by plain --retry.

Audit of the sibling pinned-binary installs in this workflow: Trivy
(aquasecurity/trivy-action), semgrep (container image), OSV-Scanner
(google/osv-scanner-action) and TruffleHog (pinned action) download nothing
with curl — the gitleaks step was the only release-tarball download in ci.yml.
Same gap exists in security-audit.yml's identical gitleaks install step
(out of this PR's stated scope; flagged for a follow-up).

Validated: actionlint clean on the workflow; run block bash -n clean.
@sonarqubecloud

sonarqubecloud Bot commented Sep 8, 2026

Copy link
Copy Markdown

@mahmutkaya
mahmutkaya merged commit b3960e1 into main Sep 8, 2026
70 checks 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.

1 participant