Potential fix for code scanning alert no. 1: Workflow does not contain permissions #6124
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Potential fix for https://github.com/Catrobat/Catroweb/security/code-scanning/1
In general, to fix this issue we should explicitly specify the minimal required permissions for the
GITHUB_TOKENin this workflow, either globally at the root level or per job. Since this workflow has a single job and only needs to read repository contents (foractions/checkoutand the PHP script), the token can be limited tocontents: read.The best fix without changing existing functionality is to add a
permissions:block that restrictscontentstoread. This can be added at the workflow root (right aftername:and beforeon:) so it applies to all jobs, or inside thecheck_for_new_catroid_bricksjob. Root-level is cleaner and future-proof for additional jobs. No imports or other code changes are needed; we only adjust the YAML to include:Concretely, in
.github/workflows/check_for_new_bricks.yaml:permissions:section after line 13 (name: Check for new bricks) and before line 15 (# This check is independent...).No other lines or behavior need to be modified.
Suggested fixes powered by Copilot Autofix. Review carefully before merging.