http client dependency cleanup and fine tuning connection performance

Author: ibalosh

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 1.10.21
+
+* updated default HTTP client connection settings
+* adjusted HTTP client library package used
+
 ## 1.10.2
 
 * updated HTTP client data requests issue with unicode characters

pom.xml

@@ -23,7 +23,7 @@
     </developers>
 
     <properties>
-        <postmark.version>1.10.2</postmark.version>
+        <postmark.version>1.10.21</postmark.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <java.version>1.8</java.version>
         <jackson.minimum.version>2.9.7</jackson.minimum.version>
@@ -50,7 +50,7 @@
 
         <dependency>
             <groupId>org.apache.httpcomponents.client5</groupId>
-            <artifactId>httpclient5-fluent</artifactId>
+            <artifactId>httpclient5</artifactId>
             <version>5.2.1</version>
         </dependency>
 

src/main/java/com/postmarkapp/postmark/client/HttpClient.java

@@ -3,6 +3,7 @@
 import org.apache.hc.client5.http.config.RequestConfig;
 import org.apache.hc.client5.http.impl.classic.CloseableHttpClient;
 import org.apache.hc.client5.http.impl.classic.HttpClientBuilder;
+import org.apache.hc.client5.http.impl.io.PoolingHttpClientConnectionManager;
 import org.apache.hc.core5.http.ClassicHttpRequest;
 import org.apache.hc.core5.http.ContentType;
 import org.apache.hc.core5.http.io.entity.EntityUtils;
@@ -35,6 +36,8 @@ public enum DEFAULTS {
     // client configuration options like timeouts, forwarding ..
     private RequestConfig.Builder clientConfigBuilder;
 
+    private final PoolingHttpClientConnectionManager connectionManager;
+
     private boolean secureConnection = true;
 
     public HttpClient(Map<String,Object> headers, int connectTimeoutSeconds, int readTimeoutSeconds) {
@@ -44,6 +47,10 @@ public HttpClient(Map<String,Object> headers, int connectTimeoutSeconds, int rea
                 .setConnectTimeout(Timeout.ofSeconds(connectTimeoutSeconds))
                 .setResponseTimeout(Timeout.ofSeconds(readTimeoutSeconds));
 
+        this.connectionManager = new PoolingHttpClientConnectionManager();
+        connectionManager.setMaxTotal(100);
+        connectionManager.setDefaultMaxPerRoute(25);
+
         this.client = buildClient();
     }
 
@@ -144,7 +151,11 @@ public CloseableHttpClient getClient() {
      * @return initialized HTTP client
      */
     private CloseableHttpClient buildClient() {
-        return HttpClientBuilder.create().setDefaultRequestConfig(clientConfigBuilder.build()).build();
+        return HttpClientBuilder
+                .create()
+                .setDefaultRequestConfig(clientConfigBuilder.build())
+                .setConnectionManager(connectionManager)
+                .build();
     }
 
     /**