Winhttp flaky test

sdk/core/azure-core/CHANGELOG.md

@@ -8,6 +8,8 @@
 
 ### Bugs Fixed
 
+- Fixed and issue where WinHttp transport was not closing correctly in case of request timeout.
+
 ### Other Changes
 
 - Move the connection back to the connection pool when HTTP error 404 was received. This may improve the performance of a multithreaded application when libcurl transport adapter is being used. (A community contribution, courtesy of _[mchelnokov](https://github.com/mchelnokov)_)

sdk/core/azure-core/src/http/winhttp/win_http_request.hpp

@@ -178,7 +178,7 @@ namespace Azure { namespace Core { namespace Http { namespace _detail {
         WinHttpTransportOptions const& options);
 
     ~WinHttpRequest();
-
+    void MarkRequestHandleClosed() { m_requestHandleClosed = true; };
     void Upload(Azure::Core::Http::Request& request, Azure::Core::Context const& context);
     void SendRequest(Azure::Core::Http::Request& request, Azure::Core::Context const& context);
     void ReceiveResponse(Azure::Core::Context const& context);

sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp

@@ -528,17 +528,15 @@ namespace Azure { namespace Core { namespace Http { namespace _detail {
       // destroyed *after* lock is destroyed, ensuring that the event is not set to the signalled
       // state before the lock is released.
       auto scope_exit{m_actionCompleteEvent.SetEvent_scope_exit()};
+      std::unique_lock<std::mutex> lock(m_actionCompleteMutex);
+      m_stowedErrorInformation = stowedErrorInformation;
+      m_stowedError = stowedError;
     }
     else
     {
       Log::Write(
-          Logger::Level::Verbose,
-          "Received error while closing: " + std::to_string(stowedError));
+          Logger::Level::Verbose, "Received error while closing: " + std::to_string(stowedError));
     }
-
-    std::unique_lock<std::mutex> lock(m_actionCompleteMutex);
-    m_stowedErrorInformation = stowedErrorInformation;
-    m_stowedError = stowedError;
   }
 
   DWORD WinHttpAction::GetStowedError()
@@ -574,10 +572,9 @@ namespace Azure { namespace Core { namespace Http { namespace _detail {
     {
       return;
     }
-
+    WinHttpAction* httpAction = reinterpret_cast<WinHttpAction*>(dwContext);
     try
     {
-      WinHttpAction* httpAction = reinterpret_cast<WinHttpAction*>(dwContext);
       httpAction->OnHttpStatusOperation(
           hInternet, internetStatus, statusInformation, statusInformationLength);
     }
@@ -588,6 +585,7 @@ namespace Azure { namespace Core { namespace Http { namespace _detail {
           Logger::Level::Error,
           "Request Failed Exception Thrown: " + std::string(rfe.what()) + rfe.Message);
       WinHttpCloseHandle(hInternet);
+      httpAction->m_httpRequest->MarkRequestHandleClosed();
     }
     catch (std::exception const& ex)
     {