Skip to content

fix: add -r flag to jq in SBOM digest extraction#630

Merged
dottorblaster merged 1 commit into
rancher-sandbox:mainfrom
dottorblaster:fix-jq-attestation-quotes
May 18, 2026
Merged

fix: add -r flag to jq in SBOM digest extraction#630
dottorblaster merged 1 commit into
rancher-sandbox:mainfrom
dottorblaster:fix-jq-attestation-quotes

Conversation

@dottorblaster
Copy link
Copy Markdown
Member

What this PR does / why we need it:

Without -r, jq emits a JSON-encoded string with literal quotes around the digest, which get propagated through $GITHUB_OUTPUT into the downstream crane blob call. This PR fixes it.

Which issue(s) this PR fixes

fixes #

Special notes for your reviewer:

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

Signed-off-by: Alessio Biancalana <alessio.biancalana@suse.com>
@dottorblaster dottorblaster self-assigned this May 18, 2026
@dottorblaster dottorblaster added the bug Something isn't working label May 18, 2026
@github-project-automation github-project-automation Bot moved this to Pending Review in Runtime Enforcer May 18, 2026
Copy link
Copy Markdown
Collaborator

@Andreagit97 Andreagit97 left a comment

Choose a reason for hiding this comment

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

thanks

@Andreagit97 Andreagit97 added this to the 0.6.0 milestone May 18, 2026
@dottorblaster dottorblaster enabled auto-merge May 18, 2026 14:24
@dottorblaster dottorblaster merged commit 2cc4b45 into rancher-sandbox:main May 18, 2026
13 checks passed
@github-project-automation github-project-automation Bot moved this from Pending Review to Done in Runtime Enforcer May 18, 2026
@dottorblaster dottorblaster deleted the fix-jq-attestation-quotes branch May 18, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants