fix(security): add explicit workflow permissions#671
fix(security): add explicit workflow permissions#671cl-efornaciari wants to merge 2 commits intodevelopfrom
Conversation
Resolves CodeQL actions/missing-workflow-permissions alerts. Adds explicit permissions blocks to all workflow YAML files.
|
👋 cl-efornaciari, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
chainchad
left a comment
There was a problem hiding this comment.
Nit but you could add:
permissions: {}
At the top-level and then set each jobs permissions with exactly what it needs including contents: read
|
|
This PR is stale because it has been open 30 days with no activity. |
|
This PR has been automatically closed because it has been stale for > 30 days. |





Summary
Adds explicit
permissionsblocks to all workflow YAML files to resolve CodeQLactions/missing-workflow-permissionsalerts (~12 alerts).Changes
permissions: { contents: read }to static-analysis, sonar-scan, lint, relayer, integration_gauntlet, integration-tests-soak, integration-tests-smoke, integration-tests-publish, monitoring-build-push-ecr, golangci-lint, examples, contractscontents: readand job-levelcontents: writefor starknet-relayer and starknet-gauntlet-cli release workflowscontents: write,pull-requests: write,actions: write(required for creating release PRs and dispatching workflows)Scope
Workflow-only changes. Code alerts are handled in a separate PR.
Made with Cursor