Skip to content

Comments

The req documentation incorrectly states that we default to md5 (1.1.0)#6905

Closed
mattcaswell wants to merge 1 commit intoopenssl:OpenSSL_1_1_0-stablefrom
mattcaswell:md5-not-the-default
Closed

The req documentation incorrectly states that we default to md5 (1.1.0)#6905
mattcaswell wants to merge 1 commit intoopenssl:OpenSSL_1_1_0-stablefrom
mattcaswell:md5-not-the-default

Conversation

@mattcaswell
Copy link
Member

Just remove that statement. It's not been true since 2005.

This is for 1.1.0. It is fixed by #6901 on master.

Just remove that statement. It's not been true since 2005.
@richsalz richsalz added the approval: done This pull request has the required number of approvals label Aug 9, 2018
@mattcaswell mattcaswell added this to the Assessed milestone Aug 20, 2018
@mattcaswell mattcaswell removed the approval: done This pull request has the required number of approvals label Sep 3, 2018
@mattcaswell
Copy link
Member Author

I need another reviewer on this...anyone?

Also the related #6906.

Copy link
Member

@romen romen left a comment

Choose a reason for hiding this comment

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

LGTM

@mattcaswell
Copy link
Member Author

Pushed. Thanks.

@mattcaswell mattcaswell closed this Sep 4, 2018
levitte pushed a commit that referenced this pull request Sep 4, 2018
Just remove that statement. It's not been true since 2005.

Reviewed-by: Nicola Tuveri <[email protected]>
(Merged from #6905)
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.

3 participants