Skip to content

chore: pin GitHub Actions to immutable SHAs#32

Merged
KooshaPari merged 1 commit into
mainfrom
chore/20260430-pin-actions-v2
Apr 30, 2026
Merged

chore: pin GitHub Actions to immutable SHAs#32
KooshaPari merged 1 commit into
mainfrom
chore/20260430-pin-actions-v2

Conversation

@KooshaPari

@KooshaPari KooshaPari commented Apr 30, 2026

Copy link
Copy Markdown
Owner

Pin GitHub Actions to immutable SHA references for supply chain security.

🤖 Generated with Claude Code


Note

Medium Risk
Mostly CI/workflow hardening by pinning actions/checkout, but it also adds large generated SBOM JSON files that embed absolute local file paths, which could create noise or unintended info exposure if published/used downstream.

Overview
Pins GitHub Actions checkout to an immutable commit SHA (actions/checkout@b4ffde… / v4.1.1) across all workflows to reduce supply-chain risk.

Adds generated CycloneDX SBOM outputs for crates/pheno-ffi-go (crates_pheno-ffi-go.json and crates_pheno-ffi-python.json), introducing large tracked artifacts that enumerate dependencies and include build metadata (e.g., target triple and file URLs).

Reviewed by Cursor Bugbot for commit 8c3cb99. Bugbot is set up for automated code reviews on this repo. Configure here.

@KooshaPari KooshaPari merged commit 3cd67f7 into main Apr 30, 2026
1 check failed
@gemini-code-assist

Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@KooshaPari KooshaPari deleted the chore/20260430-pin-actions-v2 branch April 30, 2026 18:13

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.

Reviewed by Cursor Bugbot for commit 8c3cb99. Configure here.

"ref": "registry+https://github.com/rust-lang/crates.io-index#zmij@1.0.21"
}
]
} No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

SBOM file has wrong name, contains Go data

Medium Severity

The file crates/pheno-ffi-go/crates_pheno-ffi-python.json is named "python" but its metadata.component describes pheno-ffi-go with "Go FFI bindings for phenotype-config". Its content is nearly identical to crates/pheno-ffi-go/crates_pheno-ffi-go.json (same component, different UUID/timestamp). Meanwhile, crates/pheno-ffi-python/crates_pheno-ffi-go.json contains actual Python SBOM data. The filenames appear swapped between the two crate directories. Any tooling consuming these SBOMs will get incorrect dependency information.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 8c3cb99. Configure here.

],
"component": {
"type": "library",
"bom-ref": "path+file:///Users/kooshapari/CodeProjects/Phenotype/repos/Configra/crates/pheno-ffi-go#0.1.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

SBOM files leak developer local filesystem paths

Low Severity

Both new SBOM files contain absolute local filesystem paths like path+file:///Users/kooshapari/CodeProjects/Phenotype/repos/Configra/... in multiple bom-ref, purl, and ref fields throughout the files. This exposes the developer's username and directory structure in committed artifacts, which is an information disclosure concern — especially notable in a PR focused on improving supply chain security.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 8c3cb99. Configure here.

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.

1 participant