From e6f5a4401b9b0d3706c6a6a2bfd4bb25da2db401 Mon Sep 17 00:00:00 2001
From: wanghaemq <wanghamax@gmail.com>
Date: Mon, 20 Mar 2023 19:27:20 +0800
Subject: [PATCH] * FIX [quic_api] Add some comments to some quic options. QUIC
 TLS cacertificate is supported.

---
 src/mqtt/protocol/mqtt/mqtt_quic.c   | 16 +++++------
 src/supplemental/quic/CMakeLists.txt | 43 +++++++---------------------
 src/supplemental/quic/quic_api.c     |  1 -
 3 files changed, 18 insertions(+), 42 deletions(-)

diff --git a/src/mqtt/protocol/mqtt/mqtt_quic.c b/src/mqtt/protocol/mqtt/mqtt_quic.c
index 703d8189..980c516c 100644
--- a/src/mqtt/protocol/mqtt/mqtt_quic.c
+++ b/src/mqtt/protocol/mqtt/mqtt_quic.c
@@ -112,7 +112,7 @@ struct conf_bridge_node {
 static conf_bridge_node config_node = {
 	.tls = {
 		.enable = false,
-		.url    = "",
+		.url    = "", // Depracated
 		.cafile = "",
 		.certfile = "",
 		.keyfile  = "",
@@ -120,18 +120,18 @@ static conf_bridge_node config_node = {
 		.cert     = "",
 		.key      = "",
 		.key_password = "",
-		.verify_peer = false,
-		.set_fail = false,
+		.verify_peer = true,
+		.set_fail = true,
 	},
-	.multi_stream = true,
-	.stream_auto_genid = true,
-	.qos_first = true,
-	.hybrid = false,
+	.multi_stream = false,
+	.stream_auto_genid = false,
+	.qos_first = false,
+	.hybrid = false, // Depracated
 	.qkeepalive = 30,
 	.qconnect_timeout = 60,
 	.qdiscon_timeout = 30,
 	.qidle_timeout = 30,
-	.qcongestion_control = 1, // bbr
+	.qcongestion_control = 0, // cubic
 	.max_send_queue_len = 32,
 	.max_recv_queue_len = 32,
 };
diff --git a/src/supplemental/quic/CMakeLists.txt b/src/supplemental/quic/CMakeLists.txt
index 2f182935..23665c34 100644
--- a/src/supplemental/quic/CMakeLists.txt
+++ b/src/supplemental/quic/CMakeLists.txt
@@ -16,39 +16,16 @@ if (NNG_ENABLE_QUIC)
     nng_sources(quic_api.c)
     nng_sources(quic_api.h)
 
-    find_library(OPENSSLQUIC_LIBRARIES
-    NAMES openssl ssl libssl libopenssl
-    HINTS "${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl"
-    PATHS "${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl"
-    PATH_SUFFIXES lib)
-
-find_library(CRYPTOQUIC_LIBRARIES
-    NAMES crypto libcrypto
-    HINTS "${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl"
-    PATHS "${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl"
-    PATH_SUFFIXES lib)
-
- if(OPENSSLQUIC_LIBRARIES_NOTFOUND)
-     message("No openssl for MsQuic be found!")
- elif(CRYPTOQUIC_LIBRARIES_NOTFOUND)
-     message("No openssl for MsQuic be found!")
- else()
-     nng_include_directories(${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl/include)
-
-     set(SSLQUIC_LIBRARIES ${OPENSSLQUIC_LIBRARIES})
-     mark_as_advanced(SSLQUIC_LIBRARIES)
-     nng_link_libraries(${SSLQUIC_LIBRARIES})
-
-     set(SSLCRYPTO_LIBRARIES ${CRYPTOQUIC_LIBRARIES})
-     mark_as_advanced(CRYPTOQUIC_LIBRARIES)
-     nng_link_libraries(${CRYPTOQUIC_LIBRARIES})
- endif()
-
- #nng_include_directories(${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl/include)
- #nng_link_libraries("${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl/lib/libssl.a")
- #nng_link_libraries("${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl/lib/libcrypto.a")
-
-
+    # In sdk. use openssl from system
+    find_package(OpenSSL REQUIRED)
+    if(OPENSSL_FOUND)
+        include_directories(${OPENSSL_INCLUDE_DIRS})
+        nng_link_libraries(OpenSSL::SSL OpenSSL::Crypto)
+    endif()
+
+    #nng_include_directories(${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl/include)
+    #nng_link_libraries("${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl/lib/libssl.a")
+    #nng_link_libraries("${CMAKE_SOURCE_DIR}/build/_deps/opensslquic-build/openssl/lib/libcrypto.a")
 
     # set_property(CACHE NNG_QUIC_LIB PROPERTY STRINGS ${NNG_TLS_ENGINES})
 else ()
diff --git a/src/supplemental/quic/quic_api.c b/src/supplemental/quic/quic_api.c
index 4a208137..cb0e09ad 100644
--- a/src/supplemental/quic/quic_api.c
+++ b/src/supplemental/quic/quic_api.c
@@ -1482,7 +1482,6 @@ void
 quic_proto_set_sdk_config(void *config)
 {
 	memcpy(&conf_node, config, sizeof(conf_quic_sdk));
-	log_info("-------------------------------------tls %d", conf_node.tls.enable);
 }
 
 void