Skip to content

Comments

fix: remove pull_request_target security vulnerability#22

Merged
airtonix merged 3 commits intochore/allow-patch-before-majorfrom
copilot/sub-pr-20
Nov 19, 2025
Merged

fix: remove pull_request_target security vulnerability#22
airtonix merged 3 commits intochore/allow-patch-before-majorfrom
copilot/sub-pr-20

Conversation

Copy link
Contributor

Copilot AI commented Nov 19, 2025

The PR workflow used pull_request_target with explicit checkout of untrusted PR code (ref: ${{ github.event.pull_request.head.sha }}), allowing malicious PRs to execute arbitrary code with write permissions.

Changes

  • Changed workflow trigger from pull_request_target to pull_request
  • Removed explicit ref parameter from checkout action
  • Retained pull-requests: write permission for semantic PR action

The workflow now executes PR code in a restricted context without elevated privileges while maintaining full functionality.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: airtonix <61225+airtonix@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP to allow pre-major versions to bump patch fix: remove pull_request_target security vulnerability Nov 19, 2025
Copilot AI requested a review from airtonix November 19, 2025 10:23
@airtonix airtonix marked this pull request as ready for review November 19, 2025 11:29
@airtonix airtonix merged commit 12dec83 into chore/allow-patch-before-major Nov 19, 2025
1 check passed
@airtonix airtonix deleted the copilot/sub-pr-20 branch November 19, 2025 11: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