Skip to content

chore(ci): harden CI workflows - #17

Open
amagan wants to merge 1 commit into
developfrom
fix/harden-ci-security
Open

chore(ci): harden CI workflows#17
amagan wants to merge 1 commit into
developfrom
fix/harden-ci-security

Conversation

@amagan

@amagan amagan commented Aug 14, 2026

Copy link
Copy Markdown
Contributor
  • Add least-privilege permissions: blocks to CI workflows.
  • Pin actions/create-github-app-token and actions/setup-java to commit SHAs.
  • Run PR verification for external contributors.

No source, dependency or runtime changes.

…g, open PR checks)

F-CI-001: remove the repo-admin gate from code-npm_node-PR_verify.yml so external contributors' fork PRs can run read-only lint/test/build. The job runs on pull_request (no secret exposure) with permissions: contents: read.

F-SEC-003: add least-privilege permissions to PR_verify, PR_verify-fallback and the four release_preview jobs (clears the missing-workflow-permissions alerts); SHA-pin actions/create-github-app-token@v3 and actions/setup-java@v5.

Note: the issue_comment '/publish-snapshot' publisher (F-SEC-001) is intentionally left untouched here; it will be redesigned separately (untrusted build vs trusted publish, immutable artifact hand-off, environment with required reviewers).
@amagan
amagan requested a review from a team as a code owner August 14, 2026 13:04
@sonarqubecloud

Copy link
Copy Markdown

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