Repository: knox
Updated Branches:
  refs/heads/master 1526f58b2 -> cb6bbb582


KNOX-1533 - Disable httpclient transparent content decompression

Signed-off-by: Kevin Risden <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/knox/repo
Commit: http://git-wip-us.apache.org/repos/asf/knox/commit/cb6bbb58
Tree: http://git-wip-us.apache.org/repos/asf/knox/tree/cb6bbb58
Diff: http://git-wip-us.apache.org/repos/asf/knox/diff/cb6bbb58

Branch: refs/heads/master
Commit: cb6bbb5821fbbfaf344fbb7364749cd4c1888e2c
Parents: 1526f58
Author: Kevin Risden <[email protected]>
Authored: Fri Oct 19 14:39:51 2018 -0400
Committer: Kevin Risden <[email protected]>
Committed: Tue Oct 30 13:51:54 2018 -0400

----------------------------------------------------------------------
 .../knox/gateway/dispatch/DefaultHttpClientFactory.java   | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/knox/blob/cb6bbb58/gateway-spi/src/main/java/org/apache/knox/gateway/dispatch/DefaultHttpClientFactory.java
----------------------------------------------------------------------
diff --git 
a/gateway-spi/src/main/java/org/apache/knox/gateway/dispatch/DefaultHttpClientFactory.java
 
b/gateway-spi/src/main/java/org/apache/knox/gateway/dispatch/DefaultHttpClientFactory.java
index 619a04d..29a66de 100644
--- 
a/gateway-spi/src/main/java/org/apache/knox/gateway/dispatch/DefaultHttpClientFactory.java
+++ 
b/gateway-spi/src/main/java/org/apache/knox/gateway/dispatch/DefaultHttpClientFactory.java
@@ -113,11 +113,11 @@ public class DefaultHttpClientFactory implements 
HttpClientFactory {
           .register(AuthSchemes.SPNEGO, new KnoxSpnegoAuthSchemeFactory(true))
           .build();
 
-      builder = builder.setDefaultAuthSchemeRegistry(authSchemeRegistry)
+      builder.setDefaultAuthSchemeRegistry(authSchemeRegistry)
           .setDefaultCookieStore(new HadoopAuthCookieStore(gatewayConfig))
           .setDefaultCredentialsProvider(credentialsProvider);
     } else {
-      builder = builder.setDefaultCookieStore(new NoCookieStore());
+      builder.setDefaultCookieStore(new NoCookieStore());
     }
 
     builder.setKeepAliveStrategy( DefaultConnectionKeepAliveStrategy.INSTANCE 
);
@@ -131,8 +131,10 @@ public class DefaultHttpClientFactory implements 
HttpClientFactory {
 
     builder.setDefaultRequestConfig( getRequestConfig( filterConfig ) );
 
-    HttpClient client = builder.build();
-    return client;
+    // See KNOX-1530 for details
+    builder.disableContentCompression();
+
+    return builder.build();
   }
 
   private static RequestConfig getRequestConfig( FilterConfig config ) {

Reply via email to