Closed andrewhop closed 1 month ago
Attention: Patch coverage is 26.66667%
with 11 lines
in your changes missing coverage. Please review.
Project coverage is 78.31%. Comparing base (
c9f6b6d
) to head (97a4dfb
).
Files | Patch % | Lines |
---|---|---|
tool-openssl/x509_test.cc | 11.11% | 8 Missing :warning: |
tool-openssl/x509.cc | 50.00% | 3 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description of changes:
Add the -text option to the X509 tool which prints a certificate's contents in a human readable format. Also add the version tool which only supports printing the basic version information.
Testing:
Added a new test for X509 -text with some minor caveats due to difference in between OpenSSL versions. I didn't add a test for the version as that will always be different:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.