-
Notifications
You must be signed in to change notification settings - Fork 558
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update verify-blob to support DSSEs #1355
Conversation
cc @priyawadhwa |
Using the signature and the envelope I put in the issue, I used it like that:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding this @rgreinho ! Just added one comment about unit testing :)
2049796
to
c925381
Compare
Adds the ability to verify DSSEs from the `verify-blob` command. Fixes sigstore#1321 Signed-off-by: Rémy Greinhofer <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @rgreinho !
Adds the ability to verify DSSEs from the `verify-blob` command. Fixes sigstore#1321 Signed-off-by: Rémy Greinhofer <[email protected]>
Adds the ability to verify DSSEs from the
verify-blob
command.Fixes #1321