Make druid client follow redirects by default (#214)

yahoo · Mar 31, 2017 · fa5a4e2 · fa5a4e2
1 parent 6a0418c
commit fa5a4e2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -133,6 +133,9 @@ Current
 
 ### Fixed:
 
+- [Default the AsyncDruidWebServiceImpl to follow redirects](https://github.com/yahoo/fili/pull/214)
+    * It defaulted to not following redirects, and now it doesn't
+
 - [Reenable custom query types in TestDruidWebService]()
 
 - [Fixed Segment Metadata Loader Unconfigured Dimension Bug](https://github.com/yahoo/fili/pull/197)

diff --git a/...e/src/main/java/com/yahoo/bard/webservice/druid/client/impl/AsyncDruidWebServiceImpl.java b/...e/src/main/java/com/yahoo/bard/webservice/druid/client/impl/AsyncDruidWebServiceImpl.java
@@ -115,6 +115,7 @@ private static AsyncHttpClient initializeWebClient(int requestTimeout) {
                 .setRequestTimeout(requestTimeout)
                 .setConnectionTtl(requestTimeout)
                 .setPooledConnectionIdleTimeout(requestTimeout)
+                .setFollowRedirect(true)
                 .build();
 
         return new DefaultAsyncHttpClient(config);