Skip to content

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

Merged
rm3l merged 1 commit into
redhat-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 into
redhat-developer:release-1.7from
openshift-cherrypick-robot:cherry-pick-223-to-release-1.7

Conversation

@openshift-cherrypick-robot

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

Copy link
Copy Markdown

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

@rm3l rm3l left a comment

Copy link
Copy Markdown
Member

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