Skip to content

(fix)[CI] - Allow release-assets.githubusercontent.com in harden runner#782

Merged
christophetd merged 1 commit intomainfrom
simon.marechal/harden-runner-allow-gh-assets
Mar 25, 2026
Merged

(fix)[CI] - Allow release-assets.githubusercontent.com in harden runner#782
christophetd merged 1 commit intomainfrom
simon.marechal/harden-runner-allow-gh-assets

Conversation

@Minosity-VR
Copy link
Copy Markdown
Collaborator

@Minosity-VR Minosity-VR commented Mar 24, 2026

What does this PR do?

Bug Fix:

The staticcheck CI job was failing with exit code 22 (HTTP error) during Go setup: https://github.com/DataDog/stratus-red-team/actions/runs/23483213009/job/68332033526?pr=781#step:10:73

The Harden Runner post-job audit log shows that curl was blocked attempting to reach release-assets.githubusercontent.com.
This domain is called by WillAbides/setup-go-faster (used internally by dominikh/staticcheck-action) to download the Go toolchain binary. My guess is that GitHub silently migrated release asset downloads from objects.githubusercontent.com to release-assets.githubusercontent.com at the CDN level, so the same action code now resolves to a different domain at runtime. It would match this article by the team behind the harden runner: https://www.stepsecurity.io/blog/harden-runner-detects-new-traffic-to-release-assets-githubusercontent-com-across-multiple-customers (but 1 year late 🤷)

This PR will have to be force-merged because it won't pass CI as the code used for the test is the code on main and not the code in the branch

Merging this, then rebasing #781 should be enough to fix the CI issue and unblock the other PR

@Minosity-VR Minosity-VR requested review from a team as code owners March 24, 2026 10:15
@christophetd christophetd merged commit 18ffc80 into main Mar 25, 2026
5 of 7 checks passed
@christophetd christophetd deleted the simon.marechal/harden-runner-allow-gh-assets branch March 25, 2026 12:36
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