Fixing Boost.Asio based http_client to not use client_config timeout value for determining timeout on unused sockets.

Author: stgates

Release/include/cpprest/http_client.h

@@ -82,7 +82,7 @@ class http_client_config
 public:
     http_client_config() :
         m_guarantee_order(false),
-        m_timeout(utility::seconds(30)),
+        m_timeout(std::chrono::seconds(30)),
         m_chunksize(0)
 #if !defined(__cplusplus_winrt)
         , m_validate_certificates(true)

Release/src/http/client/http_client_asio.cpp

@@ -305,10 +305,10 @@ class asio_client : public _http_client_communicator, public std::enable_shared_
 {
 public:
     asio_client(http::uri address, http_client_config client_config)
-    : _http_client_communicator(std::move(address), client_config)
+    : _http_client_communicator(std::move(address), std::move(client_config))
     , m_pool(crossplat::threadpool::shared_instance().service(),
              base_uri().scheme() == "https",
-             client_config.timeout())
+             std::chrono::seconds(30)) // Unused sockets are kept in pool for 30 seconds.
     , m_resolver(crossplat::threadpool::shared_instance().service())
     {}