-
Couldn't load subscription status.
- Fork 100
fix: change timestamp validation to return new required info fields #1191
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
Conversation
Timestamp must be validated before using for validation Timestamp trust list checks Support intermediate certs trust anchors for OpenSSL path Support separate trust list Fix for upcoming spec change to disallow empty algorithm value in a Claim
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1191 +/- ##
==========================================
- Coverage 79.57% 79.09% -0.48%
==========================================
Files 147 147
Lines 40551 41009 +458
==========================================
+ Hits 32268 32438 +170
- Misses 8283 8571 +288 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Short note and link - The checks on the PR for clippy are failing due to new enforced clippy rules after the Rust update. |
…bugs Fix OCSP responses Cleanup in Store for verify_async
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.
I left a couple of comments and an idea for better logging context.
Changes in this pull request
Log info for every timestamp error
Timestamp must be validated against Timestamp trust before using
Support intermediate certs trust anchors for OpenSSL path
Support separate trust list reporting
Fix for upcoming spec change to disallow empty algorithm value in a Claim
Dump name of cert that was validated
Checklist
TO DOitems (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.