Skip to content

Mitigate pull_request_target privilege escalation#7616

Merged
andrewrabert merged 1 commit intomasterfrom
fix/replace-pull-request-target
Feb 20, 2026
Merged

Mitigate pull_request_target privilege escalation#7616
andrewrabert merged 1 commit intomasterfrom
fix/replace-pull-request-target

Conversation

@andrewrabert
Copy link
Member

Hotfix: replace pull_request_target with pull_request to stop granting write permissions and secret access to fork PRs.

Some workflows will break (PR comments, labeling, deploy previews). Can be restored later with workflow_run.

https://securitylab.github.com/resources/github-actions-preventing-pwn-requests/

See also: #7555

Hotfix — replaces pull_request_target with pull_request to stop
granting write permissions and secrets to fork PRs. Some workflows
will break; can be fixed properly later.
@andrewrabert andrewrabert requested a review from a team as a code owner February 20, 2026 04:55
@sonarqubecloud
Copy link

@jellyfin-bot
Copy link
Collaborator

jellyfin-bot commented Feb 20, 2026

Cloudflare Pages deployment

Latest commit bcd2d11
Status ✅ Deployed!
Preview URL https://c91c82f3.jellyfin-web.pages.dev
Type 🔀 Preview

View build logs

@andrewrabert andrewrabert merged commit c9d70d7 into master Feb 20, 2026
13 checks passed
@andrewrabert andrewrabert deleted the fix/replace-pull-request-target branch February 20, 2026 04:58
@andrewrabert
Copy link
Member Author

This was a hole that needed to be addressed. I took the blind sed s/pull_request_target/pull_request/g approach to just get it done now rather than finding the time to properly fix the pipelines.

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.

3 participants