校验三次证书不是bug

Author: dayu521

main.cpp

@@ -19,6 +19,7 @@
 #include <spdlog/cfg/env.h>
 
 #include <thread>
+#include <cassert>
 
 #include "httpserver.h"
 #include "dns.h"
@@ -112,8 +113,9 @@ int main(int argc, char **argv)
                 spdlog::debug("线程{}退出成功", i);
             }
         };
+        auto core_size=std::thread::hardware_concurrency()%32;
         // 为了防止对象在多线程情况下销毁出问题
-        std::jthread t(create_thread, create_thread, 1);
+        std::jthread t(create_thread, create_thread, core_size);
 
         io_context.run();
     }

src/https/mitm_svc.cpp

@@ -69,9 +69,10 @@ namespace hcpp
                         }
                         // 可以根据需要处理其他类型的Subject Alternative Name
                     }
+                    //XXX 一般服务器会发送证书链,包含多个证书,也就是校验函数会被调用多次
+                    //我们只对这些有主体名(SAN)的证书创建假证书
+                    pci.pubkey_ = make_pem_str(X509_get_X509_PUBKEY(cert));
                 }
-
-                pci.pubkey_ = make_pem_str(X509_get_X509_PUBKEY(cert));
                 return true;
             }
             else
@@ -81,7 +82,7 @@ namespace hcpp
             //  X509_NAME_oneline(X509_get_subject_name(cert), subject_name, sizeof(subject_name)); });
         };
     }
-    
+
     awaitable<void> mitm_svc::make_memory(std::string svc_host, std::string svc_service)
     {
         try
@@ -105,7 +106,6 @@ namespace hcpp
                     {
                         if (verify_fun)
                         {
-                            log::info("mitm_svc::make_memory: 校验{}开始", host);
                             return verify_fun(preverified, v_ctx);
                         }
                         return true;

test/ssl/ssl_handshake.cpp

@@ -1,4 +1,6 @@
-// BUG 复现三次校验证书
+// XXX 复现三次校验证书
+// 注意 这不是bug,因为服务器会发送不止一个证书
+// 可能设置其他校验规则,例如,校验深度 会改变校验三次的情况
 
 #include <asio/ssl.hpp>
 #include <asio/ip/tcp.hpp>
@@ -34,7 +36,7 @@ struct endpoint
 inline std::optional<endpoint> check_http_url(std::string url)
 {
     // thread_local static std::regex re{R"(^https?://([^/]+)/.*)"};
-    thread_local static std::regex re{R"((https?)://([^/]+)(?::(\d+))?(/.*)?)"};
+    thread_local static std::regex re{R"((https?)://([^/:]+)(?::(\d+))?(/.*)?)"};
     std::smatch sm;
     endpoint ep{};
     if (std::regex_search(url, sm, re))