Skip to content

Allow unsafe PR checkout in build-gate-approved workflows - #1595

Merged
ehl-jf merged 1 commit into
masterfrom
JGC-534-allow-unsafe-pr-checkout
Jul 28, 2026
Merged

Allow unsafe PR checkout in build-gate-approved workflows#1595
ehl-jf merged 1 commit into
masterfrom
JGC-534-allow-unsafe-pr-checkout

Conversation

@ehl-jf

@ehl-jf ehl-jf commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Same root cause as jfrog-cli: actions/checkout@v7's default fork-checkout refusal under pull_request_target broke frogbot-scan-pull-request.yml and test.yml. Both are only reachable through build-gate.yml's approval gate, so adding allow-unsafe-pr-checkout: true there is safe — a maintainer already reviews the diff before approving. dependabot-auto-merge.yml's checkout is left untouched since it isn't behind that gate.

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • All static analysis checks passed.
  • This pull request is on the master branch.
  • I used gofmt for formatting the code before submitting the pull request.

@ehl-jf ehl-jf added the ignore for release Automatically generated release notes label Jul 28, 2026
@ehl-jf
ehl-jf deployed to build-gate July 28, 2026 12:31 — with GitHub Actions Active
@ehl-jf
ehl-jf enabled auto-merge (squash) July 28, 2026 12:34
@github-actions

Copy link
Copy Markdown
Contributor

👍 Frogbot scanned this pull request and did not find any new security issues.


@ehl-jf
ehl-jf merged commit 34b27f0 into master Jul 28, 2026
13 checks passed
@ehl-jf
ehl-jf deleted the JGC-534-allow-unsafe-pr-checkout branch July 28, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore for release Automatically generated release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants