Skip to content

Fix action permissions. - #24

Merged
llamavert merged 3 commits into
mainfrom
alert-autofix-2
Jul 14, 2025
Merged

Fix action permissions.#24
llamavert merged 3 commits into
mainfrom
alert-autofix-2

Conversation

@AussieScorcher

@AussieScorcher AussieScorcher commented Jul 13, 2025

Copy link
Copy Markdown
Member

This pull request includes updates to the GitHub Actions workflows to enhance security and permissions management. The changes ensure that workflows operate with the minimum necessary permissions.

Updates to GitHub Actions workflows:

  • .github/workflows/ci.yml: Added permissions block with contents: read to restrict access to repository contents during the CI workflow.
  • .github/workflows/deploy.yml: Added permissions block with contents: read to restrict access to repository contents during the deploy workflow.

…n permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@AussieScorcher
AussieScorcher marked this pull request as ready for review July 13, 2025 20:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Adds a root-level permissions block to the CI workflow, restricting the GITHUB_TOKEN to read-only access for repository contents.

  • Introduce permissions: contents: read at the top of the workflow to satisfy the least-privilege requirement.

…n permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@AussieScorcher AussieScorcher changed the title Potential fix for code scanning alert no. 2: Workflow does not contain permissions Fix action permissions. Jul 13, 2025
@BenWalker01

Copy link
Copy Markdown

Not particularly pertinent to this issue, but is there a reason you use two different versions of actions/checkout between the 2 workflows?

@AussieScorcher

Copy link
Copy Markdown
Member Author

@BenWalker01 There is no reason, I just didn't notice haha, will be updating now :)

@llamavert
llamavert merged commit 277f317 into main Jul 14, 2025
@llamavert
llamavert deleted the alert-autofix-2 branch July 14, 2025 03:49
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.

4 participants