Skip to content

NRS-4588: feed attest-sbom from the secret-free subjects list - #350

Merged
CybotTM merged 1 commit into
mainfrom
fix/attest-subjects
Aug 6, 2026
Merged

NRS-4588: feed attest-sbom from the secret-free subjects list#350
CybotTM merged 1 commit into
mainfrom
fix/attest-subjects

Conversation

@CybotTM

@CybotTM CybotTM commented Aug 6, 2026

Copy link
Copy Markdown
Member

GitHub drops a job output wholesale when any part of it matches a secret (annotation: "Skip output 'metadata' since it may contain secret") — and raw bake metadata carries the build definition including tokens interpolated into build args. On netresearch/typo3-demo run 31057574829 this left attest-sbom with an empty bake-metadata input: prepare parsed zero subjects and the attest matrix died at startup, invisible in the job rollup (all listed jobs green, run red). build-container-bake.yml now exposes its filtered sbom-subjects list (name+digest only, secret-free by construction) as a workflow output, and attest-sbom.yml prefers the new subjects input over bake-metadata, failing loudly when both are empty. The legacy metadata path also gets the non-object guard from #349.

GitHub drops a job output wholesale when any part of it matches a
secret ("Skip output metadata since it may contain secret") - and raw
bake metadata carries the build definition incl. tokens interpolated
into build args. On netresearch/typo3-demo run 31057574829 that left
attest-sbom with an empty bake-metadata input: the prepare job parsed
zero subjects and the attest matrix died at startup, invisible in the
job rollup. build-container-bake.yml now exposes its filtered
sbom-subjects list (name+digest only) as a workflow output and
attest-sbom.yml prefers a new subjects input over bake-metadata,
erroring loudly when both are empty.

Signed-off-by: Sebastian Mendel <sebastian.mendel@netresearch.de>
Copilot AI lite review requested due to automatic review settings August 6, 2026 00:09
@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@CybotTM
CybotTM merged commit 9e8fbcb into main Aug 6, 2026
12 of 13 checks passed
@CybotTM
CybotTM deleted the fix/attest-subjects branch August 6, 2026 00:12
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