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
Add padding spaces before printing signature algorithm. #1513
Conversation
Is there any new progress on this PR? |
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.
ping for another reviewer.
@hotpxl do you agree this is a trivial change? |
Is there a formal definition of trivial change? If not, then yes. This only affects human-readable text output. It doesn't involve any security related features. |
Sorry I wasn't clear. Something small, simple, and/or obvious that is therefore not copyrightable code. |
Yes, definitely. |
ping @paulidale to confirm triviality :) |
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.
Agree trivial
CLA: trivial Reviewed-by: Paul Dale <paul.dale@oracle.com> Reviewed-by: Rich Salz <rsalz@openssl.org> Reviewed-by: Rich Salz <tjh@openssl.org> (Merged from #1513)
Thanks. Merged to master. (And thanks for the nudge, this had been forgotten, sorry.) |
CLA: trivial Reviewed-by: Paul Dale <paul.dale@oracle.com> Reviewed-by: Rich Salz <rsalz@openssl.org> Reviewed-by: Rich Salz <tjh@openssl.org> (Merged from openssl#1513)
When printing x509 certificates like
It has two
Signature Algorithm
lines. InCHANGES:7466
it introduces a functionX509_signature_print
to deduplicate the print routine. But it would result in one of the lines having incorrect indentation.This PR manually added the indentation back.