Skip to content

[release-1.7] ci: Fix authorization logic in pull_request_target workflows#224

Merged
rm3l merged 1 commit intoredhat-developer:release-1.7from
openshift-cherrypick-robot:cherry-pick-223-to-release-1.7
Aug 27, 2025
Merged

[release-1.7] ci: Fix authorization logic in pull_request_target workflows#224
rm3l merged 1 commit intoredhat-developer:release-1.7from
openshift-cherrypick-robot:cherry-pick-223-to-release-1.7

Conversation

@openshift-cherrypick-robot
Copy link
Copy Markdown

@openshift-cherrypick-robot openshift-cherrypick-robot commented Aug 27, 2025

This is an automated cherry-pick of #223

/assign rm3l

Summary by Sourcery

Fix authorization logic and checkout ref in pull_request_target pre-commit workflow

Bug Fixes:

  • Determine internal/external environment using pull_request.user.login rather than github.actor for accurate authorization
  • Use pull_request.head.sha instead of pull_request.head.ref to checkout the exact commit SHA

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

@rm3l rm3l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Aug 27, 2025
@rm3l rm3l merged commit a244610 into redhat-developer:release-1.7 Aug 27, 2025
7 checks passed
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.

2 participants