Skip to content

[discussion] Add harden-runner (audit mode) to release job, ahead of signing pipeline restoration - #13

Merged
mohitbansal-webflow merged 1 commit into
mainfrom
discussion/harden-runner-audit-release
Oct 16, 2026
Merged

mohitbansal-webflow merged 1 commit into
mainfrom
discussion/harden-runner-audit-release

Conversation

@mohitbansal-webflow

Copy link
Copy Markdown
Contributor

Per ACTION_PIN_POLICY.md's "ask first" guidance — requesting an exception to add step-security/harden-runner to the allowlist.

What: Egress-audit visibility (never blocking) on publish-source-release, the one job here holding a contents: write token behind an environment gate.

Why now, not later: release.yml's header notes the cosign/SBOM/attestation pipeline is pending restoration (blocked on macOS notarization). That pipeline will introduce real secrets — Apple signing/notarization credentials — into this exact job. Wiring in audit-mode monitoring now means it's already in place and has already accumulated baseline data by the time those credentials land, instead of being a rushed addition alongside the signing work itself.

Why audit, not block: Block mode requires a maintained egress allowlist; flipping it on before we know the signing pipeline's real network footprint risks breaking a release on a missed domain. Audit-only, at least for now, changes nothing functionally — pure observability.

Why scoped to one job: harden-runner runs a privileged eBPF-based agent, not a passive action. lint/test/build/codeql/vuln-scan/scorecard never hold a secret worth watching — adding it there is attack surface and an external-service dependency for no offsetting benefit.

Note: free tier (public repo, GitHub-hosted runners) covers audit mode with no ambiguity. If we ever consider egress-policy: block, that tier question should be reconfirmed directly with StepSecurity before committing — I found conflicting info on whether block mode is free vs. Enterprise-gated and don't want to assert either way.

cc @webflow/infrastructure-security

@mohitbansal-webflow
mohitbansal-webflow marked this pull request as ready for review October 15, 2026 22:20
Allowlist step-security/harden-runner and wire it into
publish-source-release as an audit-only first step, ahead of the
pending cosign/SBOM/notarization pipeline restoration that will bring
real signing credentials into this job. Scoped to this one job only;
see .github/action-allowlist.txt for the rationale.
@mohitbansal-webflow
mohitbansal-webflow force-pushed the discussion/harden-runner-audit-release branch from 5bf1e5b to 0c40e06 Compare October 16, 2026 03:54
@mohitbansal-webflow
mohitbansal-webflow merged commit 4f771bd into main Oct 16, 2026
14 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.

2 participants