From 54ae35740eeac40e30b5ca9e799bb8995a6e4612 Mon Sep 17 00:00:00 2001 From: Victor Vazquez Date: Wed, 13 Apr 2022 17:49:55 +0000 Subject: [PATCH] revert setting WINHTTP_NO_CLIENT_CERT_CONTEXT --- .../src/http/winhttp/win_http_transport.cpp | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp b/sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp index f82fd79c8..1f7e9b947 100644 --- a/sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp +++ b/sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp @@ -342,23 +342,6 @@ void WinHttpTransport::CreateRequestHandle(std::unique_ptr<_detail::HandleManage GetErrorAndThrow("Error while getting a request handle."); } - if (requestSecureHttp) - { - // If the service requests TLS client certificates, we want to let the WinHTTP APIs know that - // it's ok to initiate the request without a client certificate. - // - // Note: If/When TLS client certificate support is added to the pipeline, this line may need to - // be revisited. - if (!WinHttpSetOption( - handleManager->m_requestHandle, - WINHTTP_OPTION_CLIENT_CERT_CONTEXT, - WINHTTP_NO_CLIENT_CERT_CONTEXT, - 0)) - { - GetErrorAndThrow("Error while setting client cert context to ignore.."); - } - } - if (m_options.IgnoreUnknownCertificateAuthority) { auto option = SECURITY_FLAG_IGNORE_UNKNOWN_CA;