Update qcom-preflight-checks to latest.yml#86
Update qcom-preflight-checks to latest.yml#86Sandhya1236 wants to merge 1 commit intoqualcomm-linux:video.qclinux.0.0from
Conversation
pull_request_target can be a security vulnerability and should be avoided and also aligned with latest pre-flight check workflow. Signed-off-by: Sandhya Adavikolanu <sadaviko@qti.qualcomm.com>
f57ce66 to
d4e3ae5
Compare
|
CI failure is expected on fork PRs with pull_request This job is failing at actions/checkout (before QC Preflight runs). The workflow is attempting to fetch the PR head repo (fork) and checkout fails with “Repository not found” / exit code 128 because the workflow token can’t access the fork in our internal/private setup. This matches OSDO guidance in “OSS Ops Guidance: https://github.qualcomm.com/OSDO/osdo.github.qualcomm.com/pull/212/files Prefer splitting untrusted and trusted workflows: |
Running untrusted code on the pull_request_target trigger may lead to security vulnerabilities. These vulnerabilities include cache poisoning and granting unintended access to write privileges or secrets.
https://docs.github.com/en/actions/reference/workflows-and-actions/events-that-trigger-workflows#pull_request_target
We should update all usage of pull_request_target in all workflow files and also update qualcomm-preflight-check to the latest.