Skip to content

Add fingerprint text, remove MD5 #4906

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.

Sign up for GitHub

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

Jump to bottom
Closed
richsalz wants to merge 1 commit into from
Closed

Add fingerprint text, remove MD5 #4906

richsalz wants to merge 1 commit into from

Conversation

@richsalz
Copy link
Contributor

@richsalz richsalz commented Dec 11, 2017

Fixes: #3509

@richsalz richsalz added branch: 1.0.2 Applies to OpenSSL_1_0_2-stable branch (EOL) 1.1.0 branch: master Applies to master branch approval: review pending This pull request needs review by a committer labels Dec 11, 2017
@richsalz richsalz self-assigned this Dec 11, 2017
kaduk
kaduk approved these changes Dec 11, 2017
View reviewed changes
Copy link
Contributor

@kaduk kaduk left a comment

Choose a reason for hiding this comment

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

For some reason I was surprised to see the 1.0.2 tag, but I do not object to it.

@levitte levitte added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Dec 11, 2017
levitte pushed a commit that referenced this pull request Jan 7, 2018
Add fingerprint text, remove MD5
9422d45 
Reviewed-by: Ben Kaduk <kaduk@mit.edu>
(Merged from #4906)
levitte pushed a commit that referenced this pull request Jan 7, 2018
Add fingerprint text, remove MD5
794bf5f 
Reviewed-by: Ben Kaduk <kaduk@mit.edu>
(Merged from #4906)
(Cherry-picked from commit 9422d45)
levitte pushed a commit that referenced this pull request Jan 7, 2018
Add fingerprint text, remove MD5
a353f94 
Reviewed-by: Ben Kaduk <kaduk@mit.edu>
(Merged from #4906)
(cherry picked from commit 794bf5f)
@richsalz
Copy link
Contributor Author

richsalz commented Jan 7, 2018

Merged to all three branches, thanks.

@richsalz richsalz closed this Jan 7, 2018
@richsalz richsalz deleted the gh3509 branch January 7, 2018 03:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@kaduk kaduk kaduk approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@richsalz richsalz

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch branch: 1.0.2 Applies to OpenSSL_1_0_2-stable branch (EOL)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@richsalz @kaduk @levitte