Skip to content

Potential fix for code scanning alert no. 210: Workflow does not contain permissions#1098

Merged
michaelhortongsa merged 2 commits into
mainfrom
merge-schedule-permission-fix
May 13, 2025
Merged

Potential fix for code scanning alert no. 210: Workflow does not contain permissions#1098
michaelhortongsa merged 2 commits into
mainfrom
merge-schedule-permission-fix

Conversation

@drewnielson
Copy link
Copy Markdown
Contributor

Potential fix for https://github.com/GSA/Section508.gov/security/code-scanning/210

To fix the issue, we will add a permissions block at the workflow level to explicitly define the least privileges required. Based on the workflow's functionality, it likely needs contents: read to access repository contents and pull-requests: write to manage pull requests (e.g., merging them). These permissions will be added to the root of the workflow to apply to all jobs.


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

drewnielson and others added 2 commits May 9, 2025 13:03
…ain permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@drewnielson drewnielson marked this pull request as ready for review May 13, 2025 13:26
@michaelhortongsa michaelhortongsa merged commit 44ca78d into main May 13, 2025
5 checks passed
@michaelhortongsa michaelhortongsa deleted the merge-schedule-permission-fix branch May 13, 2025 13:29
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