Skip to content

[CI][Security] ROCM-26563: enforce HTTPS verification in PSDB trigger#3086

Open
vkallesh wants to merge 1 commit into
amd-stagingfrom
amd/dev/vkallesh/ROCM-26563-enforce-https
Open

[CI][Security] ROCM-26563: enforce HTTPS verification in PSDB trigger#3086
vkallesh wants to merge 1 commit into
amd-stagingfrom
amd/dev/vkallesh/ROCM-26563-enforce-https

Conversation

@vkallesh

Copy link
Copy Markdown
Collaborator

Fix for the issue "ROCM-26563 — PSDB Jenkins trigger: HTTPS verification disabled"

End-to-end flow: A PR to amd-staging runs the GitHub Actions workflow PSDB-amd-staging.yml on a self-hosted runner. It pulls an internal Docker image (JENKINS_TRIGGER_DOCKER_IMAGE) and runs two Python scripts inside it — cancel_previous_build.py and jenkins_api.py — which call the Jenkins server and the GitHub API to cancel stale builds and trigger the PSDB pipeline.

Issue: Both scripts were launched with PYTHONHTTPSVERIFY=0, which globally disables TLS certificate verification. This left the Jenkins and GitHub calls open to man-in-the-middle attacks and credential interception (CI_JENKINS_TOKEN, GitHub PAT). The flag is unnecessary — Jenkins presents a valid public DigiCert certificate.

… workflow

Remove PYTHONHTTPSVERIFY=0 from the cancel_previous_build.py and jenkins_api.py
docker exec invocations in PSDB-amd-staging.yml. This flag globally disabled
Python TLS certificate verification for all outbound HTTPS in those processes
(Jenkins API and GitHub API calls), exposing credentials (CI_JENKINS_TOKEN,
GITHUB_PAT) to man-in-the-middle attacks. As the current Jenkins endpoint
presents a valid, publicly-trusted DigiCert certificate, so verification can
be safely enforced.
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.

2 participants