Skip to content

Github Workflow does not contain permissions stanza#253

Merged
rocktavious merged 3 commits into
mainfrom
kr/permissions-fixes
Aug 8, 2025
Merged

Github Workflow does not contain permissions stanza#253
rocktavious merged 3 commits into
mainfrom
kr/permissions-fixes

Conversation

@rocktavious
Copy link
Copy Markdown
Contributor

Potential fix for https://github.com/OpsLevel/opslevel-runner/security/code-scanning/5

To fix the issue, we will add a permissions block at the root of the workflow file. This block will define the minimal permissions required for the workflow to function securely. Since the workflow uses an external action (opslevel/actions/.github/workflows/grype.yml@main), we will assume it primarily needs read access to repository contents and no write permissions unless explicitly required. The permissions block will be added immediately after the name field.


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

rocktavious and others added 2 commits July 16, 2025 12:15
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@rocktavious rocktavious changed the title Potential fix for code scanning alert no. 5: Workflow does not contain permissions Github Workflow does not contain permissions stanza Jul 16, 2025
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@rocktavious rocktavious self-assigned this Jul 16, 2025
@rocktavious rocktavious marked this pull request as ready for review July 16, 2025 17:19
@jasonopslevel
Copy link
Copy Markdown
Contributor

sorry - i missed this @rocktavious

@rocktavious rocktavious merged commit 59113a1 into main Aug 8, 2025
5 checks passed
@rocktavious rocktavious deleted the kr/permissions-fixes branch August 8, 2025 18:55
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.

2 participants