From f6420979b286429b6d64bcee8c597b4a39050960 Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 16 Oct 2026 19:39:00 +0000 Subject: [PATCH] Drop the redundant has_verify_callback flag from the wolfSSL context It only mirrored whether verify_callback was set, so test verify_callback directly, as the Mbed TLS context now does. Co-Authored-By: Claude Sonnet 5.5 Claude-Session: https://claude.ai/code/session_01WGAYpQgXt95LdDQcJYgjB8 --- httplib.h | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/httplib.h b/httplib.h index 8c37dea2..ee9e8bc6 100644 --- a/httplib.h +++ b/httplib.h @@ -4039,7 +4039,6 @@ struct WolfSSLContext { WOLFSSL_CTX *ctx = nullptr; bool is_server = false; bool verify_client = false; - bool has_verify_callback = false; // Set by tls::set_verify_callback(). VerifyCallback verify_callback; std::string ca_pem_data_; // accumulated PEM for get_ca_names/get_ca_certs @@ -21965,9 +21964,9 @@ inline void set_verify_client(ctx_t ctx, bool require) { if (require) { wolfSSL_CTX_set_verify( wctx->ctx, SSL_VERIFY_PEER | SSL_VERIFY_FAIL_IF_NO_PEER_CERT, - wctx->has_verify_callback ? impl::wolfssl_verify_callback : nullptr); + wctx->verify_callback ? impl::wolfssl_verify_callback : nullptr); } else { - if (wctx->has_verify_callback) { + if (wctx->verify_callback) { wolfSSL_CTX_set_verify(wctx->ctx, SSL_VERIFY_PEER, impl::wolfssl_verify_callback); } else { @@ -22725,9 +22724,8 @@ inline bool set_verify_callback(ctx_t ctx, VerifyCallback callback) { auto *wctx = static_cast(ctx); wctx->verify_callback = std::move(callback); - wctx->has_verify_callback = static_cast(wctx->verify_callback); - if (wctx->has_verify_callback) { + if (wctx->verify_callback) { wolfSSL_CTX_set_verify(wctx->ctx, SSL_VERIFY_PEER, impl::wolfssl_verify_callback); } else {