Bump apache http library version and remove deprecated stale connecti…

…on check configuration
rtveitch · Apr 28, 2015 · 49f5272 · 49f5272
1 parent cc2c37f
commit 49f5272
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/og-client/src/main/java/com/cleversafe/og/client/ApacheClient.java b/og-client/src/main/java/com/cleversafe/og/client/ApacheClient.java
@@ -181,12 +181,9 @@ public Double read(JsonReader in) throws IOException {
                 new CustomHttpRequestRetryHandler(this.retryCount, this.requestSentRetry))
             .setRedirectStrategy(new CustomRedirectStrategy())
             .setDefaultRequestConfig(
-                RequestConfig.custom()
-                    .setExpectContinueEnabled(this.expectContinue)
-                    // TODO investigate performance impact of stale check (30ms reported)
-                    .setStaleConnectionCheckEnabled(true).setRedirectsEnabled(true)
-                    .setRelativeRedirectsAllowed(true).setConnectTimeout(this.connectTimeout)
-                    .setSocketTimeout(this.soTimeout)
+                RequestConfig.custom().setExpectContinueEnabled(this.expectContinue)
+                    .setRedirectsEnabled(true).setRelativeRedirectsAllowed(true)
+                    .setConnectTimeout(this.connectTimeout).setSocketTimeout(this.soTimeout)
                     // TODO should this be infinite? length of time allowed to request a connection
                     // from the pool
                     .setConnectionRequestTimeout(0).build()).build();

diff --git a/pom.xml b/pom.xml
@@ -189,7 +189,7 @@
       <dependency>
         <groupId>org.apache.httpcomponents</groupId>
         <artifactId>httpclient</artifactId>
-        <version>4.3.5</version>
+        <version>4.4.1</version>
       </dependency>
       <dependency>
         <groupId>com.github.tomakehurst</groupId>