Skip to content

Potential fix for code scanning alert no. 1: Workflow does not contain permissions#61

Merged
SoloByte merged 1 commit intomainfrom
workflow-permission-fix
May 27, 2025
Merged

Potential fix for code scanning alert no. 1: Workflow does not contain permissions#61
SoloByte merged 1 commit intomainfrom
workflow-permission-fix

Conversation

@SoloByte
Copy link
Collaborator

Potential fix for https://github.com/DaveGreen-Games/ShapeEngine/security/code-scanning/1

To fix the issue, we will add a permissions block at the root of the workflow file. This block will specify the minimal permissions required for the workflow to function correctly. Based on the steps in the workflow, the contents: read permission is sufficient, as the workflow only checks out the repository, restores dependencies, builds the project, and uploads artifacts. No write permissions are needed.

The permissions block will be added immediately after the name field in the workflow file.


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

…n permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@SoloByte SoloByte marked this pull request as ready for review May 27, 2025 14:32
@SoloByte SoloByte merged commit 7ef2eb4 into main May 27, 2025
7 checks passed
@SoloByte SoloByte deleted the workflow-permission-fix branch May 27, 2025 14:33
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