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

configure: add warning for using TLS libraries without 1.3 support #12900

Closed
wants to merge 2 commits into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Feb 8, 2024

BearSSL, mbedTLS and Secure Transport

@vszakats

This comment was marked as outdated.

@vszakats
Copy link
Member

vszakats commented Feb 8, 2024

Similar for CMake:

--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -720,6 +720,12 @@ if(USE_MSH3)
   list(APPEND CURL_LIBS ${MSH3_LIBRARIES})
 endif()
 
+if(USE_MBEDTLS OR
+   USE_BEARSSL OR
+   USE_SECTRANSP)
+  message(WARNING "A selected TLS library does not support TLS 1.3.")
+endif()
+
 if(CURL_WITH_MULTI_SSL AND (USE_NGTCP2 OR USE_QUICHE OR USE_MSH3))
   message(FATAL_ERROR "MultiSSL cannot be enabled with HTTP/3 and vice versa.")
 endif()

@bagder bagder closed this in 05104f8 Feb 9, 2024
@bagder bagder deleted the bagder/configure-warn-tl13 branch February 9, 2024 15:03
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.

2 participants