Skip to content

/curl

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

openssl: adapt to 3.0.0 #3462

Closed
wants to merge 3 commits into from

Conversation

@bagder
Copy link
Member

commented Jan 11, 2019

OpenSSL_version() replaces OpenSSL_version_num()
@bagder
openssl: adapt to 3.0.0 
OpenSSL_version() replaces OpenSSL_version_num()
Verified
This commit was signed with a verified signature.
bagder Daniel Stenberg
GPG key ID: 5CC908FDB71E12C2 Learn about signing commits
Loading status checks…
39fab90

@bagder bagder added the SSL/TLS label Jan 11, 2019

bagder added 2 commits Jan 11, 2019
@bagder
fixup the #elif logic
Verified
This commit was signed with a verified signature.
bagder Daniel Stenberg
GPG key ID: 5CC908FDB71E12C2 Learn about signing commits
Loading status checks…
4dc7cd4
@bagder
fixup check the define too
Verified
This commit was signed with a verified signature.
bagder Daniel Stenberg
GPG key ID: 5CC908FDB71E12C2 Learn about signing commits
Loading status checks…
b418e6b
@danielgustafsson
danielgustafsson reviewed Jan 12, 2019
View changes
Copy link
Member

left a comment

I'd move the comment, but the code LGTM
lib/vtls/openssl.c Show resolved Hide resolved

@bagder bagder closed this in cf8c705 Jan 13, 2019

@bagder bagder deleted the bagder/openssl-3_0_0 branch Jan 13, 2019

@lock lock bot locked as resolved and limited conversation to collaborators Apr 13, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@danielgustafsson danielgustafsson

Assignees
No one assigned
Labels
SSL/TLS
Projects
None yet
Milestone
No milestone
2 participants
@bagder @danielgustafsson
You can’t perform that action at this time.