Skip to content

Potential fix for code scanning alert no. 6: Workflow does not contain permissions#21

Merged
eschultink merged 1 commit intomainfrom
alert-autofix-6
Feb 25, 2026
Merged

Potential fix for code scanning alert no. 6: Workflow does not contain permissions#21
eschultink merged 1 commit intomainfrom
alert-autofix-6

Conversation

@eschultink
Copy link
Member

Potential fix for https://github.com/Worklytics/datastore-backup/security/code-scanning/6

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. Since the workflow only checks out the repository and runs tests, it likely only needs contents: read permissions. This ensures that the workflow has the least privilege necessary to perform its tasks.


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>
@eschultink eschultink marked this pull request as ready for review February 9, 2026 22:45
@eschultink eschultink requested review from Copilot and jlorper February 9, 2026 22:45
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds least-privilege GitHub Actions permissions to address a code scanning alert about missing workflow permissions.

Changes:

  • Adds a root-level permissions block to the CI workflow
  • Restricts workflow token to contents: read

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@eschultink eschultink merged commit 3a10d69 into main Feb 25, 2026
4 checks passed
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.

3 participants