Skip to content

Update attestation name#1540

Merged
imjasonh merged 1 commit intogoogle:mainfrom
laurentsimon:patch-1
Jan 25, 2023
Merged

Update attestation name#1540
imjasonh merged 1 commit intogoogle:mainfrom
laurentsimon:patch-1

Conversation

@laurentsimon
Copy link
Copy Markdown
Contributor

@laurentsimon laurentsimon commented Jan 24, 2023

Fix #1539

NOTE: we will release the slsa verifier Action Installer in a few weeks, so we'll be able to remove the script that manually downloads the slsa-verifier binary.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #1540 (b205929) into main (1e09daa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1540   +/-   ##
=======================================
  Coverage   73.31%   73.31%           
=======================================
  Files         117      117           
  Lines        9017     9017           
=======================================
  Hits         6611     6611           
  Misses       1746     1746           
  Partials      660      660           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

set -euo pipefail
gh -R "$GITHUB_REPOSITORY" release download "$GITHUB_REF_NAME" -p "*.tar.gz"
gh -R "$GITHUB_REPOSITORY" release download "$GITHUB_REF_NAME" -p "attestation.intoto.jsonl"
gh -R "$GITHUB_REPOSITORY" release download "$GITHUB_REF_NAME" -p "multiple.intoto.jsonl"
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.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

An alternative to that would be to use the provenance-name output which will always be the name of the provenance artifact regardless of the input.
https://github.com/slsa-framework/slsa-github-generator/blob/main/internal/builders/generic/README.md#workflow-outputs

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.

I don't have a strong preference for the filename either way, so long as verification works. Thanks for making this change!

@imjasonh imjasonh merged commit 76bac93 into google:main Jan 25, 2023
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.

SLSA provenance verification broken for 0.13.0

5 participants