generated from sigstore/sigstore-project-template
-
Notifications
You must be signed in to change notification settings - Fork 63
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix gitsign for public Sigstore changes. (#50)
* Fix gitsign for public Sigstore changes. There were a few updates to public sigstore today, that ended up breaking gitsign. 1. Fulcio now uses an intermediary cert. TIL that DER encoding will sort values for stability, which means that the ordering of these certs can change. This PR fixes gitsign for the public instance, by restricting the certs included to just the leaf cert (which is probably okay in most cases?). 2. Rekor now uses a sharded entry ID, which breaks how we were looking up Rekor entries since the sharded entry ID didn't match the underlying UUID (even though it did). Added normalization to work for either case. Also updates rekor/cosign deps, we need rekor >= 0.5 (https://blog.sigstore.dev/sigstore-project-update-march-2022-bf34aa632388) Signed-off-by: Billy Lynch <billy@chainguard.dev>
- Loading branch information
Showing
6 changed files
with
236 additions
and
134 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.