Skip to content

ci: GitHub Actions workflow security cleanup#127

Open
emptyhammond wants to merge 3 commits into
mainfrom
worktree-fixup-workflows
Open

ci: GitHub Actions workflow security cleanup#127
emptyhammond wants to merge 3 commits into
mainfrom
worktree-fixup-workflows

Conversation

@emptyhammond
Copy link
Copy Markdown

Routine hygiene pass over the GitHub Actions workflows in this repo, addressing findings from a workflow security audit. Changes are split into three commits, one per finding type:

  • Disable credential persistence on actions/checkout steps so the default GITHUB_TOKEN is not left in the local git config after checkout.
  • Scope each job's permissions explicitly: top-level permissions: {}, with each job granted only the GITHUB_TOKEN scopes it actually needs.
  • Pin all third-party actions to commit SHAs (with the tag preserved as a comment) so an upstream tag move can't silently change what runs in CI.

No behavioural changes intended — the workflows run the same checks against the same inputs.

Set persist-credentials: false on every actions/checkout step so the
default GITHUB_TOKEN is not left in the local git config after checkout.
Add top-level permissions: {} to each workflow and grant each job only
the GITHUB_TOKEN scopes it actually needs.
Pin each action reference to a commit SHA with the tag preserved as a
comment, so an upstream tag move can't silently change what runs in CI.
@emptyhammond emptyhammond requested a review from owenpearson May 26, 2026 17:36
@owenpearson
Copy link
Copy Markdown
Member

oof, CI is broken on this repo because https://github.com/ably/specification/raw/main/textile/features.textile moved to https://github.com/ably/specification/raw/main/specifications/features.md. This repo tries to parse that document so it will take some work to fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants