Skip to content
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

curl_setup: Suppress OpenSSL 3 deprecation warnings #10543

Closed
wants to merge 1 commit into from

Conversation

jay
Copy link
Member

@jay jay commented Feb 17, 2023

  • Define OPENSSL_SUPPRESS_DEPRECATED.

OpenSSL 3 has deprecated some of the functions libcurl uses such as those with DES, MD5 and ENGINE prefix. We don't have replacements for those functions so the warnings were disabled in autotools builds, but still showed in other builds.

Closes #xxxx

- Define OPENSSL_SUPPRESS_DEPRECATED.

OpenSSL 3 has deprecated some of the functions libcurl uses such as
those with DES, MD5 and ENGINE prefix. We don't have replacements for
those functions so the warnings were disabled in autotools builds, but
still showed in other builds.

Closes #xxxx
@jay jay added the build label Feb 17, 2023
Copy link
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

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

It is also added in CMakeLists.txt that you can remove now.

@bagder bagder added the TLS label Feb 17, 2023
@jay jay closed this in cab0402 Feb 19, 2023
@jay jay deleted the ossl3_suppress_deprecate branch February 19, 2023 00:04
vszakats added a commit to curl/curl-for-win that referenced this pull request Feb 19, 2023
Redundant with 7.88.1. Now set in curl source code.

Ref: curl/curl#10543
Ref: curl/curl@cab0402
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
- Define OPENSSL_SUPPRESS_DEPRECATED.

OpenSSL 3 has deprecated some of the functions libcurl uses such as
those with DES, MD5 and ENGINE prefix. We don't have replacements for
those functions so the warnings were disabled in autotools and cmake
builds, but still showed in other builds.

Closes curl#10543
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants