-
Notifications
You must be signed in to change notification settings - Fork 91
test: OCSP E2E #1172
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
test: OCSP E2E #1172
Conversation
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1172 +/- ##
=======================================
Coverage 75.27% 75.27%
=======================================
Files 60 60
Lines 3365 3365
=======================================
Hits 2533 2533
Misses 640 640
Partials 192 192 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
shizhMSFT
left a comment
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
ghost
left a comment
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
Added OCSP E2E test cases: valid, revoked and unknown OCSP fallback to CRL test cases have been included in notaryproject#1079 Resolves notaryproject#638 Resolves notaryproject#572 --------- Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Added OCSP E2E test cases: valid, revoked and unknown OCSP fallback to CRL test cases have been included in notaryproject#1079 Resolves notaryproject#638 Resolves notaryproject#572 --------- Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Added OCSP E2E test cases: valid, revoked and unknown
OCSP fallback to CRL test cases have been included in #1079
Resolves #638
Resolves #572