-
Notifications
You must be signed in to change notification settings - Fork 90
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
openssl_csr: fix bad tests, avoid accepting invalid crl_distribution_points records #560
Conversation
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.
Looks good!
Co-authored-by: Kristian Heljas <11139388+kristianheljas@users.noreply.github.com>
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.
Thanks for the improvements @felixfontein, LGTM!
Backport to stable-1: 💚 backport PR created✅ Backport PR branch: Backported as #561 🤖 @patchback |
…points records (#560) * Improve error handling. * Remove invalid tests. * Add changelog fragment. * Fix tests. * Improve exception catching. Co-authored-by: Kristian Heljas <11139388+kristianheljas@users.noreply.github.com> * Prevent empty full_name. * Fix condition. Make sure errors are caught. * Add more checks. Co-authored-by: Kristian Heljas <11139388+kristianheljas@users.noreply.github.com> (cherry picked from commit ddfb18b)
@kristianheljas thanks for reviewing this! |
…points records (#560) (#561) * Improve error handling. * Remove invalid tests. * Add changelog fragment. * Fix tests. * Improve exception catching. Co-authored-by: Kristian Heljas <11139388+kristianheljas@users.noreply.github.com> * Prevent empty full_name. * Fix condition. Make sure errors are caught. * Add more checks. Co-authored-by: Kristian Heljas <11139388+kristianheljas@users.noreply.github.com> (cherry picked from commit ddfb18b) Co-authored-by: Felix Fontein <felix@fontein.de>
SUMMARY
Nightly CI breaks due to the stricter checks in pyca/cryptography#7710.
ISSUE TYPE
COMPONENT NAME
openssl_csr
openssl_csr_pipe