Skip to content

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

Draft
adzuci wants to merge 1 commit into
mainfrom
adzuci/alert-autofix-1
Draft

adzuci wants to merge 1 commit into
mainfrom
adzuci/alert-autofix-1

Conversation

@adzuci

@adzuci adzuci commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Potential fix for https://github.com/apolloio/apollo-io-cli/security/code-scanning/1

Add explicit permissions to .github/workflows/release.yml:

  1. Set a restrictive workflow-level default:

    • permissions: { contents: read }
    • This ensures all jobs start with minimal access.
  2. Override permissions for the build job, because Create release needs write access:

    • permissions: { contents: write }
  3. Keep update-homebrew-tap at read-only default (no extra write to current repo needed), since it uses HOMEBREW_TAP_TOKEN for PR creation in another repository.

This preserves current functionality while enforcing least privilege and satisfying CodeQL.

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>

@orca-security-us orca-security-us Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

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