Skip to content

[github] Use/require fewer permissions in the 'Bump global.json' workflow. - #25469

Merged
rolfbjarne merged 1 commit into
mainfrom
dev/rolf/workflow-bump-global-json
May 19, 2026
Merged

rolfbjarne merged 1 commit into
mainfrom
dev/rolf/workflow-bump-global-json

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member

'zizmor' reports no issues for this workflow now.

…flow.

'zizmor' reports no issues for this workflow now.
Copilot AI review requested due to automatic review settings May 19, 2026 15:36

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

This PR tightens the security posture of the bump-global-json GitHub Actions workflow by switching to a less-privileged trigger and explicitly scoping permissions, aligning with zizmor recommendations.

Changes:

  • Switch workflow trigger from pull_request_target to pull_request (opened/synchronize) and set workflow-level permissions: {}.
  • Add job-level gating to skip fork PRs and restrict execution to dotnet-maestro[bot] PRs with expected titles.
  • Pin actions/checkout to a full commit SHA and adjust checkout/ref handling to operate on the PR branch and allow pushing.

Comment thread .github/workflows/bump-global-json.yml
@rolfbjarne
rolfbjarne enabled auto-merge (squash) May 19, 2026 16:33
@rolfbjarne
rolfbjarne merged commit 49f6b83 into main May 19, 2026
13 checks passed
rolfbjarne added a commit that referenced this pull request May 21, 2026
…p global.json' workflow. (#25489)

Backport of #25469, so that the workflow works in this branch.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants