Skip to content

Comments

WIP: Document supported digest functions#4979

Closed
kroeckx wants to merge 1 commit intoopenssl:masterfrom
kroeckx:sha0
Closed

WIP: Document supported digest functions#4979
kroeckx wants to merge 1 commit intoopenssl:masterfrom
kroeckx:sha0

Conversation

@kroeckx
Copy link
Member

@kroeckx kroeckx commented Dec 26, 2017

No description provided.

=item B<md5>
=item B<md4>

MD5 Digest
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Were you planning to add a "MD4 Digest" line?

=item B<sha>

SHA Digest
SHA-0 Digest
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SHA-0 was removed a while ago and openssl(1) doesn't recognize sha as option... I mean this should be omitted, not fixed.

@dot-asm
Copy link
Contributor

dot-asm commented Dec 27, 2017

On related note one can wonder if one should add a note that some digests can be unavailable because they were disabled at compile time. Alternatively one can choose to list only those that can't be disabled [optionally with a note that they are minimally guaranteed].

@mattcaswell mattcaswell added this to the 1.1.1 milestone Jan 24, 2018
@kaduk
Copy link
Contributor

kaduk commented Feb 28, 2018

@kroeckx did you plan to push an update here?

@mattcaswell
Copy link
Member

@kroeckx this is still marked as WIP. Were you planning to do something with this, or should it be closed?

@levitte
Copy link
Member

levitte commented Apr 19, 2018

Ping. @kroeckx?
(reminder: you need to address the md4 / md5 mixup)

@kroeckx
Copy link
Member Author

kroeckx commented Apr 19, 2018 via email

@levitte
Copy link
Member

levitte commented Apr 19, 2018

Okie

@levitte
Copy link
Member

levitte commented Apr 19, 2018

Closing for the benefit of #6024

@levitte levitte closed this Apr 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants