Skip to content

Fix build: pass --disable-path-validation to oras attach#139

Merged
toddysm merged 1 commit into
mainfrom
fix/oras-attach-path-validation
Jul 2, 2026
Merged

Fix build: pass --disable-path-validation to oras attach#139
toddysm merged 1 commit into
mainfrom
fix/oras-attach-path-validation

Conversation

@toddysm

@toddysm toddysm commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Problem

The build / cssc-dashboard run 28605221190 failed for all three services at the Publish attestations as OCI 1.1 referrers step with:

Error: absolute file path detected. If it's intentional, use --disable-path-validation flag to skip this check: /tmp/tmp.XXXXXX/sbom.spdx.json

Current oras rejects the absolute temp-file path passed to oras attach. The step code was unchanged, so this is an oras path-validation behavior change.

Fix

Add --disable-path-validation to the oras attach invocation. The file lives in an intentional mktemp -d directory we control, so skipping the check is safe.

actionlint clean.

oras now rejects the absolute temp file path passed to `oras attach`
("absolute file path detected"), failing the referrer-publish step.
The path is an intentional mktemp dir we control, so disable the check.
Copilot AI review requested due to automatic review settings July 2, 2026 16:28
@toddysm toddysm added bug Something isn't working build Related to the Build stage labels Jul 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the build-cssc-dashboard GitHub Actions workflow to accommodate a behavioral change in the ORAS CLI where oras attach now rejects absolute file paths by default. It ensures the workflow can continue attaching extracted SBOM/provenance in-toto statements (stored in a mktemp -d directory) as OCI 1.1 referrers.

Changes:

  • Add --disable-path-validation to the oras attach invocation used when attaching per-platform in-toto statements.
  • Preserve the existing mktemp -d based workflow structure while unblocking the “Publish attestations as OCI 1.1 referrers” step.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@toddysm
toddysm merged commit 32cf0a0 into main Jul 2, 2026
4 checks passed
@toddysm
toddysm deleted the fix/oras-attach-path-validation branch July 2, 2026 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working build Related to the Build stage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants