Pistache HTTPS 服务器在客户端主机名验证失败时的无响应问题分析

2025-06-24 18:41:10作者：曹令琨Iris

问题背景

Pistache 是一个现代化的 C++ REST 框架，提供了 HTTP 和 HTTPS 服务器的功能。近期发现 Pistache 的 HTTPS 服务器在特定场景下会出现无响应的问题，具体表现为当客户端（如 cURL）因主机名验证失败而直接关闭 TCP 连接时，服务器会进入不可用状态。

问题现象

当使用 cURL 8.9.0 或更高版本连接 Pistache HTTPS 服务器时，如果客户端的主机名验证失败（常见于服务器证书的 CN 或 SAN 不匹配客户端请求的主机名），cURL 会直接关闭 TCP 连接而不发送 TLS close_notify 消息。此时 Pistache 服务器会出现以下症状：

后续所有客户端请求只能收到空响应
服务器应用逻辑仍能正常执行请求处理
底层 SSL_write 调用不再被执行
问题持续存在直到服务器重启

技术分析

通过深入分析 OpenSSL 的行为和 Pistache 的实现，发现问题根源在于 OpenSSL 错误队列的处理机制：

正常关闭场景（客户端发送 close_notify）：
- SSL_read 返回 <=0
- SSL_get_error 返回 SSL_ERROR_ZERO_RETURN
- 错误队列为空
- SSL_get_shutdown 返回 SSL_RECEIVED_SHUTDOWN
异常关闭场景（客户端直接断开）：
- SSL_read 返回 <=0
- SSL_get_error 返回 SSL_ERROR_SSL
- 错误队列包含 SSL_R_UNEXPECTED_EOF_WHILE_READING 错误
- SSL_get_shutdown 返回 0

根据 OpenSSL 文档明确指出："当前线程的错误队列在尝试 TLS/SSL I/O 操作前必须为空，否则 SSL_get_error 无法可靠工作"。Pistache 当前实现未正确处理错误队列，导致在异常关闭后所有后续 SSL 操作都不可靠。

解决方案

修复方案是在 Transport::handleIncoming 方法中处理 SSL_read 失败时，显式清空 OpenSSL 错误队列：

bytes = SSL_read(reinterpret_cast<SSL*>(peer->ssl()),
             buffer + totalBytes,
             static_cast<int>(Const::MaxBuffer - totalBytes));
if (bytes <= 0) {
    int ssl_get_error_res = SSL_get_error(
        reinterpret_cast<SSL*>(peer->ssl()),
        static_cast<int>(bytes));
    while (ERR_get_error() != 0); // 清空错误队列
    retry = (ssl_get_error_res == SSL_ERROR_WANT_READ);
}

验证测试

为验证修复效果，开发了专门的单元测试，通过 SSL_CTX_set_quiet_shutdown 强制客户端不发送关闭通知：

TEST(https_server_test, basic_tls_request_with_no_shutdown_from_peer) {
    // 配置客户端不发送关闭通知
    const auto sslctx_Callback = +[](CURL* /* curl */, void* sslctx, void* /* parm */) -> CURLcode {
        SSL_CTX_set_quiet_shutdown(reinterpret_cast<SSL_CTX*>(sslctx), 1);
        return CURLE_OK;
    };
    
    // 执行多次请求验证服务器稳定性
    for (const auto& req_i : { 0, 1, 2, 3 }) {
        res = curl_easy_perform(curl);
        EXPECT_EQ(res, CURLE_OK);
        EXPECT_EQ(buffer, "Hello, World!");
    }
}