Skip to content
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

Change values of Identity.Raw, add fingerprints #1628

Merged
merged 2 commits into from
Aug 15, 2023

Conversation

haydentherapper
Copy link
Contributor

Raw now contains only PKIX public keys or DER encoded certificates. The keys are extracted from minisign, pgp, and TUF verifiers.

Also added fingerprints for each verifier. Keys, certificates, and ed25519 keys from minisign are hex-encoded sha-256 digests of the raw key. SSH and PGP use their ecosystem-standard fingerprints.

Summary

Release Note

Documentation

@codecov
Copy link

codecov bot commented Aug 10, 2023

Codecov Report

Merging #1628 (ce741f1) into main (08ea39a) will increase coverage by 0.23%.
The diff coverage is 86.41%.

@@            Coverage Diff             @@
##             main    #1628      +/-   ##
==========================================
+ Coverage   66.88%   67.12%   +0.23%     
==========================================
  Files          83       83              
  Lines        8441     8490      +49     
==========================================
+ Hits         5646     5699      +53     
+ Misses       2116     2114       -2     
+ Partials      679      677       -2     
Flag Coverage Δ
e2etests 47.75% <0.00%> (-0.28%) ⬇️
unittests 48.13% <86.41%> (+0.37%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
pkg/pki/tuf/tuf.go 49.23% <54.54%> (+0.43%) ⬆️
pkg/pki/ssh/ssh.go 60.29% <72.72%> (+0.29%) ⬆️
pkg/pki/pgp/pgp.go 80.17% <88.46%> (+1.97%) ⬆️
pkg/pki/minisign/minisign.go 76.80% <100.00%> (+0.96%) ⬆️
pkg/pki/pkcs7/pkcs7.go 58.82% <100.00%> (+3.11%) ⬆️
pkg/pki/x509/x509.go 72.51% <100.00%> (+2.81%) ⬆️

... and 1 file with indirect coverage changes

cpanato
cpanato previously approved these changes Aug 10, 2023
Raw now contains only PKIX public keys or DER encoded certificates. The
keys are extracted from minisign, pgp, and TUF verifiers.

Also added fingerprints for each verifier. Keys, certificates, and
ed25519 keys from minisign are hex-encoded sha-256 digests of the raw
key. SSH and PGP use their ecosystem-standard fingerprints.

Signed-off-by: Hayden Blauzvern <hblauzvern@google.com>
Signed-off-by: Hayden Blauzvern <hblauzvern@google.com>
@bobcallaway bobcallaway merged commit 3e1715a into sigstore:main Aug 15, 2023
14 checks passed
@github-actions github-actions bot added this to the v1.2.2 milestone Aug 15, 2023
@haydentherapper haydentherapper deleted the ids-update branch August 15, 2023 15:23
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.

None yet

3 participants