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

feat: adds no cert found on sig exit code #3038

Merged
merged 1 commit into from
Jun 9, 2023

Conversation

ChrisJBurns
Copy link
Contributor

@ChrisJBurns ChrisJBurns commented Jun 8, 2023

Adds the exit code and error type for when no certificate is found on a signature is thrown during verification.

Signed-off-by: ChrisJBurns 29541485+ChrisJBurns@users.noreply.github.com

Signed-off-by: ChrisJBurns <29541485+ChrisJBurns@users.noreply.github.com>
Copy link
Contributor

@haydentherapper haydentherapper left a comment

Choose a reason for hiding this comment

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

Nice, thanks!

@codecov
Copy link

codecov bot commented Jun 8, 2023

Codecov Report

Merging #3038 (21e261b) into main (2742dce) will increase coverage by 0.00%.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##             main    #3038   +/-   ##
=======================================
  Coverage   31.07%   31.08%           
=======================================
  Files         155      155           
  Lines        9732     9739    +7     
=======================================
+ Hits         3024     3027    +3     
- Misses       6247     6250    +3     
- Partials      461      462    +1     
Impacted Files Coverage Δ
pkg/cosign/errors.go 28.57% <0.00%> (-4.77%) ⬇️
pkg/cosign/verify.go 37.22% <0.00%> (+0.03%) ⬆️
cmd/cosign/errors/exit_code_lookup.go 65.38% <50.00%> (-4.62%) ⬇️

@ChrisJBurns
Copy link
Contributor Author

Am wondering if the e2e tests are just failing due to flakiness?

@znewman01 znewman01 merged commit d0cc985 into sigstore:main Jun 9, 2023
26 checks passed
@github-actions github-actions bot added this to the v1.14.0 milestone Jun 9, 2023
@ChrisJBurns ChrisJBurns deleted the adds-error-code branch June 10, 2023 01:19
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