Skip to content

ci: fix workflow permissions and pin action versions#652

Open
John Kennedy (jkennedyvz) wants to merge 1 commit intomainfrom
ci/fix-workflow-permissions
Open

ci: fix workflow permissions and pin action versions#652
John Kennedy (jkennedyvz) wants to merge 1 commit intomainfrom
ci/fix-workflow-permissions

Conversation

@jkennedyvz
Copy link
Contributor

Improves security by adding proper permissions block and pinning GitHub Actions to specific commit SHAs.

Changes:

  • Add top-level permissions: contents: read block
  • Remove redundant job-level permissions
  • Pin actions/checkout to v4 with commit SHA (34e114876b0b11c390a56381ad16ebd13914f8d5)
  • Pin KengoTODA/actions-setup-docker-compose to v1 with commit SHA (477353946803dd64eaa44008b865b6bfc88cab4e)
  • Remove LANGSMITH_LICENSE_KEY from pull_request trigger context to prevent credential exfiltration

Security Benefits:

  • Prevents supply chain attacks via action version hijacking
  • Follows GitHub's least-privilege permissions model
  • Reduces risk of secret exposure on pull_request triggers

- Add top-level permissions block with contents: read
- Remove job-level permissions (now redundant)
- Pin actions/checkout to v4 with commit SHA
- Pin KengoTODA/actions-setup-docker-compose to v1 with commit SHA
- Remove LANGSMITH_LICENSE_KEY secret from pull_request trigger context to prevent exfiltration
@jkennedyvz John Kennedy (jkennedyvz) requested a review from a team as a code owner March 25, 2026 18:16
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