Provide a setFollowRedirect method on the builder, to enable following redirection or not on a per request base, chaining

Author: erwan

src/main/java/com/ning/http/client/Request.java

@@ -144,10 +144,17 @@ public static interface EntityWriter {
      */
     public File getFile();
 
+    /**
+     * Return the {@link followRedirects}
+     * @return the {@link followRedirects}
+     */
+    public Boolean getFollowRedirects();
+
     /**
      * Return Per request configuration.
      *
      * @return Per request configuration.
      */
     public PerRequestConfig getPerRequestConfig();
+
 }

src/main/java/com/ning/http/client/RequestBuilder.java

@@ -131,4 +131,9 @@ public RequestBuilder setProxyServer(ProxyServer proxyServer) {
     public RequestBuilder setVirtualHost(String virtualHost) {
         return super.setVirtualHost(virtualHost);
     }
+
+    @Override
+    public RequestBuilder setFollowRedirects(Boolean followRedirects) {
+        return super.setFollowRedirects(followRedirects);
+    }
 }

src/main/java/com/ning/http/client/RequestBuilderBase.java

@@ -54,6 +54,7 @@ private static final class RequestImpl implements Request {
         public ProxyServer proxyServer;
         private Realm realm;
         private File file;
+        public Boolean followRedirects;
         private PerRequestConfig perRequestConfig;
 
         public RequestImpl() {
@@ -78,6 +79,7 @@ public RequestImpl(Request prototype) {
                 this.proxyServer = prototype.getProxyServer();
                 this.realm = prototype.getRealm();
                 this.file = prototype.getFile();
+                this.followRedirects = prototype.getFollowRedirects();
                 this.perRequestConfig = prototype.getPerRequestConfig();
             }
         }
@@ -215,6 +217,10 @@ public File getFile() {
             return file;
         }
 
+        public Boolean getFollowRedirects() {
+            return followRedirects;
+        }
+
         public PerRequestConfig getPerRequestConfig() {
             return perRequestConfig;
         }
@@ -449,6 +455,11 @@ public T setRealm(Realm realm) {
         return derived.cast(this);
     }
 
+    public T setFollowRedirects(Boolean followRedirects) {
+        request.followRedirects = followRedirects;
+        return derived.cast(this);
+    }
+
     public T setPerRequestConfig(PerRequestConfig perRequestConfig) {
         request.perRequestConfig = perRequestConfig;
         return derived.cast(this);

src/main/java/com/ning/http/client/providers/NettyAsyncHttpProvider.java

@@ -773,8 +773,9 @@ public void messageReceived(ChannelHandlerContext ctx, MessageEvent e) throws Ex
                     return;
                 }
 
-                if (config.isRedirectEnabled()
-                        && (statusCode == 302 || statusCode == 301)) {
+                boolean redirectEnabled = (request.getFollowRedirects() == true
+                        || request.getFollowRedirects() == null && config.isRedirectEnabled());
+                if (redirectEnabled && (statusCode == 302 || statusCode == 301)) {
 
                     if (future.incrementAndGetCurrentRedirectCount() < config.getMaxRedirects()) {