Skip to content

ci: set explicit permissions for check-commit-messages job - #3353

Open
PandaeDo wants to merge 1 commit into
mainfrom
alert-autofix-20
Open

PandaeDo wants to merge 1 commit into
mainfrom
alert-autofix-20

Conversation

@PandaeDo

Copy link
Copy Markdown
Contributor

Potential fix for https://github.com/eclipse-score/score/security/code-scanning/20

Add an explicit permissions block to the lint-commits job in .github/workflows/on-pr.yml, using least privilege.
The best minimal fix (without changing behavior) is to set:

  • contents: read

Place it under the job definition (same level as runs-on and if), before steps, matching the style already used by fep-fcp-tests.

This documents token requirements and prevents accidental privilege expansion if repo/org defaults change later.

Suggested fixes powered by Copilot Autofix. Review carefully before merging.

@github-actions

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-3353: https://eclipse-score.github.io/score/pr-3353/

Fixes code scanning alert no. 20 (Workflow does not contain
permissions) by restricting the job token to contents: read.

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Volker Häussler <volker.haeussler@qorix.ai>
@PandaeDo PandaeDo changed the title Potential fix for code scanning alert no. 20: Workflow does not contain permissions ci: set explicit permissions for check-commit-messages job Oct 21, 2026
@PandaeDo
PandaeDo marked this pull request as ready for review October 21, 2026 11:42

This branch has not been deployed

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

Labels

None yet

Projects

Status: Backlog
Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant