block checking out fork pr for pull_request_target and workflow_run#2454

aiqiaoy · 2026-06-12T19:12:35Z

Implements https://github.com/github/c2c-actions/pull/10159

TL;DR; This PR adds a check that refuses to check out fork pull request code when the workflow trigger is either pull_request_target or workflow_run, unless the workflow author explicitly opts in via a new input allow-unsafe-pr-checkout: true.

Copilot

Pull request overview

Adds a guardrail to prevent accidentally checking out fork pull request code in privileged GitHub Actions contexts (pull_request_target and PR-triggered workflow_run), with an explicit opt-in escape hatch for advanced workflows.

Changes:

Introduces assertSafePrCheckout to detect and block unsafe fork-PR checkouts in privileged events unless allow-unsafe-pr-checkout is enabled.
Adds a new action input (allow-unsafe-pr-checkout) and threads it through input parsing and settings.
Updates documentation and adds test coverage for the new safety checks.

Show a summary per file

File	Description
src/unsafe-pr-checkout-helper.ts	New helper that blocks unsafe fork PR checkout patterns in privileged events unless opted in.
src/ref-helper.ts	Exports `fromPayload` for reuse by the new safety helper.
src/input-helper.ts	Parses `allow-unsafe-pr-checkout` and invokes the new safety assertion during input handling.
src/git-source-settings.ts	Extends settings interface with `allowUnsafePrCheckout`.
README.md	Documents the new input and its intent.
action.yml	Adds the new input to the action metadata.
test/unsafe-pr-checkout-helper.test.ts	New unit tests validating allow/refuse behavior across events and patterns.
test/input-helper.test.ts	Asserts the new setting defaults to `false`.
test/git-auth-helper.test.ts	Updates test settings object to include the new required field.
dist/index.js	Bundled output updated to include the new logic.

Copilot's findings

Tip

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

Files reviewed: 9/10 changed files
Comments generated: 4

…(#332) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [actions/checkout](https://github.com/actions/checkout) | action | major | `v6.0.3` → `v7.0.0` | --- ### Release Notes <details> <summary>actions/checkout (actions/checkout)</summary> ### [`v7.0.0`](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v700) [Compare Source](actions/checkout@v6.0.3...v7.0.0) - Block checking out fork PR for pull\_request\_target and workflow\_run by [@aiqiaoy](https://github.com/aiqiaoy) in [#2454](actions/checkout#2454) - Bump actions/publish-immutable-action from 0.0.3 to 0.0.4 in the minor-actions-dependencies group across 1 directory by [@dependabot](https://github.com/dependabot)\[bot] in [#2458](actions/checkout#2458) - Bump flatted from 3.3.1 to 3.4.2 by [@dependabot](https://github.com/dependabot)\[bot] in [#2460](actions/checkout#2460) - Bump js-yaml from 4.1.0 to 4.2.0 by [@dependabot](https://github.com/dependabot)\[bot] in [#2461](actions/checkout#2461) - Bump [@actions/core](https://github.com/actions/core) and [@actions/tool-cache](https://github.com/actions/tool-cache) and Remove uuid by [@dependabot](https://github.com/dependabot)\[bot] in [#2459](actions/checkout#2459) - upgrade module to esm and update dependencies by [@aiqiaoy](https://github.com/aiqiaoy) in [#2463](actions/checkout#2463) - Bump the minor-npm-dependencies group across 1 directory with 3 updates by [@dependabot](https://github.com/dependabot)\[bot] in [#2462](actions/checkout#2462) </details> --- ### Configuration 📅 **Schedule**: (in timezone America/New_York) - Branch creation - At any time (no schedule defined) - Automerge - At any time (no schedule defined) 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate).  Reviewed-on: https://git.greyrock.io/greyrock-labs/home-ops/pulls/332

fproulx-boostsecurity · 2026-06-18T16:44:41Z

This is awesome @aiqiaoy !

This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [actions/checkout](https://github.com/actions/checkout) | action | major | `v6` → `v7.0.0` | | [actions/checkout](https://github.com/actions/checkout) | action | major | `v6.0.2` → `v7.0.0` | --- ### Release Notes <details> <summary>actions/checkout (actions/checkout)</summary> ### [`v7.0.0`](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v700) [Compare Source](actions/checkout@v6.0.3...v7.0.0) - Block checking out fork PR for pull\_request\_target and workflow\_run by [@aiqiaoy](https://github.com/aiqiaoy) in [#2454](actions/checkout#2454) - Bump actions/publish-immutable-action from 0.0.3 to 0.0.4 in the minor-actions-dependencies group across 1 directory by [@dependabot](https://github.com/dependabot)\[bot] in [#2458](actions/checkout#2458) - Bump flatted from 3.3.1 to 3.4.2 by [@dependabot](https://github.com/dependabot)\[bot] in [#2460](actions/checkout#2460) - Bump js-yaml from 4.1.0 to 4.2.0 by [@dependabot](https://github.com/dependabot)\[bot] in [#2461](actions/checkout#2461) - Bump [@actions/core](https://github.com/actions/core) and [@actions/tool-cache](https://github.com/actions/tool-cache) and Remove uuid by [@dependabot](https://github.com/dependabot)\[bot] in [#2459](actions/checkout#2459) - upgrade module to esm and update dependencies by [@aiqiaoy](https://github.com/aiqiaoy) in [#2463](actions/checkout#2463) - Bump the minor-npm-dependencies group across 1 directory with 3 updates by [@dependabot](https://github.com/dependabot)\[bot] in [#2462](actions/checkout#2462) ### [`v6.0.3`](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v603) [Compare Source](actions/checkout@v6.0.2...v6.0.3) - Fix checkout init for SHA-256 repositories by [@yaananth](https://github.com/yaananth) in [#2439](actions/checkout#2439) - fix: expand merge commit SHA regex and add SHA-256 test cases by [@yaananth](https://github.com/yaananth) in [#2414](actions/checkout#2414) ### [`v6.0.2`](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v602) [Compare Source](actions/checkout@v6.0.1...v6.0.2) - Fix tag handling: preserve annotations and explicit fetch-tags by [@ericsciple](https://github.com/ericsciple) in [#2356](actions/checkout#2356) ### [`v6.0.1`](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v601) [Compare Source](actions/checkout@v6...v6.0.1) - Add worktree support for persist-credentials includeIf by [@ericsciple](https://github.com/ericsciple) in [#2327](actions/checkout#2327) </details> --- ### Configuration 📅 **Schedule**: (in timezone Europe/Warsaw) - Branch creation - At any time (no schedule defined) - Automerge - At any time (no schedule defined) 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate).  Reviewed-on: https://git.ajgon.casa/deedee/schemas/pulls/3

This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [actions/checkout](https://github.com/actions/checkout) | action | major | `v6` → `v7` | --- ### Release Notes <details> <summary>actions/checkout (actions/checkout)</summary> ### [`v7`](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v700) [Compare Source](actions/checkout@v6...v7) - Block checking out fork PR for pull\_request\_target and workflow\_run by [@aiqiaoy](https://github.com/aiqiaoy) in [#2454](actions/checkout#2454) - Bump actions/publish-immutable-action from 0.0.3 to 0.0.4 in the minor-actions-dependencies group across 1 directory by [@dependabot](https://github.com/dependabot)\[bot] in [#2458](actions/checkout#2458) - Bump flatted from 3.3.1 to 3.4.2 by [@dependabot](https://github.com/dependabot)\[bot] in [#2460](actions/checkout#2460) - Bump js-yaml from 4.1.0 to 4.2.0 by [@dependabot](https://github.com/dependabot)\[bot] in [#2461](actions/checkout#2461) - Bump [@actions/core](https://github.com/actions/core) and [@actions/tool-cache](https://github.com/actions/tool-cache) and Remove uuid by [@dependabot](https://github.com/dependabot)\[bot] in [#2459](actions/checkout#2459) - upgrade module to esm and update dependencies by [@aiqiaoy](https://github.com/aiqiaoy) in [#2463](actions/checkout#2463) - Bump the minor-npm-dependencies group across 1 directory with 3 updates by [@dependabot](https://github.com/dependabot)\[bot] in [#2462](actions/checkout#2462) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).  Reviewed-on: https://forgejo.internal/forgejo_admin/maximus/pulls/17 Co-authored-by: Renovate Bot <renovatebot@forgejo.internal> Co-committed-by: Renovate Bot <renovatebot@forgejo.internal>

block checking out fork pr for some events

b844733

AdnaneKhan reviewed Jun 14, 2026

View reviewed changes

Comment thread dist/index.js

aiqiaoy marked this pull request as ready for review June 15, 2026 14:13

Copilot AI review requested due to automatic review settings June 15, 2026 14:13

Copilot started reviewing on behalf of aiqiaoy June 15, 2026 14:13 View session

aiqiaoy changed the title ~~block checking out fork pr for some events~~ block checking out fork pr for pull_request_target and workflow_run Jun 15, 2026

Copilot AI reviewed Jun 15, 2026

View reviewed changes

Comment thread src/unsafe-pr-checkout-helper.ts

Comment thread src/unsafe-pr-checkout-helper.ts

Comment thread README.md Outdated

Comment thread action.yml

yaananth reviewed Jun 15, 2026

View reviewed changes

Comment thread src/unsafe-pr-checkout-helper.ts Outdated

Comment thread src/unsafe-pr-checkout-helper.ts

aiqiaoy added 3 commits June 15, 2026 16:12

address copilot and reviewer feedback

12a4897

run prettier formatting

c12eb24

build

c2edb9a

yaananth previously approved these changes Jun 15, 2026

View reviewed changes

update urls

678aa28

aiqiaoy dismissed yaananth’s stale review via 678aa28 June 15, 2026 20:07

aiqiaoy added 3 commits June 15, 2026 20:16

update readme

cb140b4

update description and url again

baaeba4

edit url one more time

921dc8d

yaananth approved these changes Jun 16, 2026

View reviewed changes

aiqiaoy merged commit f9e715a into main Jun 16, 2026
12 checks passed

aiqiaoy deleted the enforce-safe-checkout branch June 16, 2026 14:03

coderabbitai Bot mentioned this pull request Jun 18, 2026

chore(deps): update actions/checkout action to v7 stranske/trip-planner#1419

Merged

1 task

prql-bot mentioned this pull request Jun 18, 2026

chore: bump actions/checkout from 6 to 7 PRQL/prql#6021

Merged

github-actions Bot mentioned this pull request Jun 18, 2026

ci(deps): bump actions/checkout from 6.0.3 to 7.0.0 in the major group across 1 directory CopilotKit/aimock#268

Open

claude Bot mentioned this pull request Jun 18, 2026

build(deps): update actions/checkout action to v7 ncaq/nix-composite-action#61

Merged

1 task

This was referenced Jun 18, 2026

Bump actions/checkout from 6 to 7 scality/log-courier#201

Open

Bump actions/checkout from 6 to 7 labrynx/auto_comments#35

Open

This was referenced Jun 18, 2026

build(deps): update actions/checkout action to v7 ncaq/konoka#359

Merged

build(deps): update actions/checkout action to v7 ncaq/kyosei-action#165

Merged

greptile-apps Bot mentioned this pull request Jun 22, 2026

chore(deps): bump actions/checkout from 6 to 7 pentiminax/ux-datatables#230

Merged

pavlojs mentioned this pull request Jun 22, 2026

chore(deps): bump actions/checkout from 6 to 7 whiteravens20/blog#8

Merged

github-actions Bot mentioned this pull request Jun 22, 2026

chore(deps): update actions/checkout action to v7 lacolaco/blog.lacolaco.net#1840

Merged

1 task

Zigfreidish mentioned this pull request Jun 22, 2026

build(deps): bump actions/checkout from 6 to 7 in the github-actions group Keith-CY/melix#2274

Merged

George-RD mentioned this pull request Jun 22, 2026

build(deps): bump actions/checkout from 6 to 7 George-RD/mag#358

Open

This was referenced Jun 22, 2026

chore(deps): update actions packages (major) lacolaco/zenn#330

Merged

chore(deps): bump the actions-all group with 2 updates sujithq/sujithq.github.io#368

Merged

llm-gateway-ai Bot mentioned this pull request Jun 23, 2026

ci(deps): bump actions/checkout from 6 to 7 zhfeng/llm-gateway#12

Merged

This was referenced Jun 23, 2026

nightly: regen reverts Dependabot action-ref bumps (e.g. actions/checkout v7→v6) max-sixty/tend#714

Closed

tend-review: actions/checkout@v7 refuses fork-PR checkout under pull_request_target max-sixty/tend#723

Closed

This was referenced Jun 23, 2026

ci(tend-review): pin actions/checkout back to v6 for fork-PR review max-sixty/worktrunk#3176

Merged

review-runs-tracking: 2026-06 max-sixty/worktrunk#2955

Open

github-actions Bot mentioned this pull request Jun 24, 2026

chore(deps): bump the actions group with 4 updates UKGovernmentBEIS/inspect_evals#1832

Open

KacperMalachowski mentioned this pull request Jun 24, 2026

fix(ci): unblock fork PRs after actions/checkout v7 bump kyma-project/test-infra#14344

Merged

This was referenced Jun 24, 2026

build(deps): bump actions/checkout from 6.0.3 to 7.0.0 ncaq/git-hooks#164

Merged

build(deps): bump actions/checkout from 6.0.3 to 7.0.0 ncaq/renovate-config#89

Merged

yairdamri-develeap mentioned this pull request Jun 24, 2026

chore(deps): bump actions/checkout from 6.0.3 to 7.0.0 in the github-actions group develeap/terraform-provider-hyperping#149

Open

tao12345666333 mentioned this pull request Jun 24, 2026

chore(deps): bump actions/checkout from 6.0.3 to 7.0.0 Kong/kong-operator#4647

Open

github-actions Bot mentioned this pull request Jun 24, 2026

Bump the actions group with 5 updates mbianchidev/corn-test#350

Open

lumirlumir mentioned this pull request Jun 24, 2026

ci: bump actions/checkout from 6 to 7 eslint/eslint#21014

Merged

jwm4 mentioned this pull request Jun 24, 2026

chore(deps): bump actions/checkout from 6 to 7 ambient-code/agentready#504

Merged

github-actions Bot mentioned this pull request Jun 24, 2026

Build(deps): Bump actions/checkout from 6.0.3 to 7.0.0 astronomer/dag-factory#806

Open

This was referenced Jun 24, 2026

ci: opt into v7 fork-PR checkout so tend-review runs on fork PRs PRQL/prql#6038

Merged

feat: add by:name to append transform PRQL/prql#6037

Draft

This was referenced Jun 24, 2026

Bump GitHub Actions dependencies wallentx/jobscout#51

Merged

Bump GitHub Actions dependencies FarmGPU/docs#1

Open

Bump GitHub Actions dependencies FarmGPU/netbox-agent#4

Open

github-actions Bot mentioned this pull request Jun 24, 2026

Update actions/checkout action to v7 - autoclosed koki-develop/koki-develop.nvim#69

Merged

1 task

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

block checking out fork pr for pull_request_target and workflow_run#2454