work around hang issue in hyper (#1550)

As indicated in #1549, there is an issue with hyper (the underlying
layer used by reqwest) that hangs in some cases on connection pools.
This PR uses a commonly discussed workaround of setting
`pool_max_idle_per_host` to 0.

Ref: hyperium/hyper#2312

Author: demoray

sdk/core/src/http_client/reqwest.rs

@@ -1,14 +1,24 @@
-use crate::error::{ErrorKind, ResultExt};
-use crate::{Body, HttpClient, PinnedStream};
-
+use crate::{
+    error::{ErrorKind, ResultExt},
+    Body, HttpClient, PinnedStream,
+};
 use async_trait::async_trait;
 use futures::TryStreamExt;
-use std::{collections::HashMap, str::FromStr};
+use std::{collections::HashMap, str::FromStr, sync::Arc};
 
 /// Construct a new `HttpClient` with the `reqwest` backend.
-pub fn new_reqwest_client() -> std::sync::Arc<dyn HttpClient> {
+pub fn new_reqwest_client() -> Arc<dyn HttpClient> {
     log::debug!("instantiating an http client using the reqwest backend");
-    std::sync::Arc::new(::reqwest::Client::new())
+
+    // set `pool_max_idle_per_host` to `0` to avoid an issue in the underlying
+    // `hyper` library that causes the `reqwest` client to hang in some cases.
+    //
+    // See <https://github.com/hyperium/hyper/issues/2312> for more details.
+    let client = ::reqwest::ClientBuilder::new()
+        .pool_max_idle_per_host(0)
+        .build()
+        .expect("failed to build `reqwest` client");
+    Arc::new(client)
 }
 
 #[cfg_attr(target_arch = "wasm32", async_trait(?Send))]