Skip to content

ci(security): add explicit permissions to all workflow jobs#58

Merged
cosmocoder merged 1 commit intomainfrom
fix/security-actions-missing-workflow-permissions
Mar 15, 2026
Merged

ci(security): add explicit permissions to all workflow jobs#58
cosmocoder merged 1 commit intomainfrom
fix/security-actions-missing-workflow-permissions

Conversation

@cosmocoder
Copy link
Copy Markdown
Owner

Add explicit permissions to all jobs in the release workflow that were missing them.

Severity: Medium
Rule: actions/missing-workflow-permissions

Alerts resolved:

  • #3 — .github/workflows/release.yml:52 (lint)
  • #4 — .github/workflows/release.yml:78 (lint-report)
  • #5 — .github/workflows/release.yml:88 (test)
  • #6 — .github/workflows/release.yml:121 (coverage)
  • #8 — .github/workflows/release.yml:150 (test-report)

Changes:
Added least-privilege permissions to five workflow jobs: contents: read for jobs that need checkout and cache access (lint, test, coverage), and empty permissions (permissions: {}) for report jobs that need no repository access (lint-report, test-report).

- Add contents: read to lint, test, and coverage jobs that need
  repository checkout and cache access
- Add empty permissions to lint-report and test-report jobs that
  need no repository access
- Follows least-privilege principle for GitHub Actions workflows
@cosmocoder cosmocoder marked this pull request as ready for review March 15, 2026 19:45
@cosmocoder cosmocoder merged commit 1e056a5 into main Mar 15, 2026
14 checks passed
@cosmocoder cosmocoder deleted the fix/security-actions-missing-workflow-permissions branch March 15, 2026 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant