-
Notifications
You must be signed in to change notification settings - Fork 8
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
Improve help message #33
Conversation
Thank you also for this PR. Hints on which MAC algorithms are available would have been found in the man page or online documentation,
I am not sure if the link https://raw.githubusercontent.com/openssl/openssl/master/include/openssl/obj_mac.h that you suggested to add would be sufficiently simple to use and provides better info than the text given there:
Moreover, the CLI help output needs to be very brief and should not have overlong lines.
|
@DDvO I updated my PR based on your proposal. Please double check. But honestly, I don't find the man-page helpful. Take for example Why is one algorithm called |
Looks good - nice that you added it for both
This is most likely due to the legacy and wonders of IETF & IANA standardization.
something like
|
@DDvO please double check again. |
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 - thanks for contributing.
Motivation
Improve documentation. Took us ages to find the list of allowed algorithms
Proposed Changes
Link to openssl header file in which the algorithms are defined.
Test Plan
n/a