Backport AsyncHttpClient#391

Author: Stephane Landelle

src/main/java/com/ning/http/client/providers/apache/ApacheResponse.java

@@ -146,7 +146,16 @@ public FluentCaseInsensitiveStringsMap getHeaders() {
     /* @Override */
 
     public boolean isRedirected() {
-        return (status.getStatusCode() >= 300) && (status.getStatusCode() <= 399);
+        switch (status.getStatusCode()) {
+        case 301:
+        case 302:
+        case 303:
+        case 307:
+        case 308:
+            return true;
+        default:
+            return false;
+        }
     }
 
     /* @Override */

src/main/java/com/ning/http/client/providers/grizzly/GrizzlyResponse.java

@@ -248,9 +248,16 @@ public FluentCaseInsensitiveStringsMap getHeaders() {
      * {@inheritDoc}
      */
     public boolean isRedirected() {
-
-        return between(status.getStatusCode(), 300, 399);
-
+        switch (status.getStatusCode()) {
+        case 301:
+        case 302:
+        case 303:
+        case 307:
+        case 308:
+            return true;
+        default:
+            return false;
+        }
     }
 
 
@@ -345,14 +352,4 @@ private Charset getCharset(final String charset) {
         return Charsets.lookupCharset(charsetLocal);
 
     }
-
-
-    private boolean between(final int value,
-                            final int lowerBound,
-                            final int upperBound) {
-
-        return (value >= lowerBound && value <= upperBound);
-
-    }
-
 }

src/main/java/com/ning/http/client/providers/jdk/JDKResponse.java

@@ -161,7 +161,16 @@ public FluentCaseInsensitiveStringsMap getHeaders() {
     /* @Override */
 
     public boolean isRedirected() {
-        return (status.getStatusCode() >= 300) && (status.getStatusCode() <= 399);
+        switch (status.getStatusCode()) {
+        case 301:
+        case 302:
+        case 303:
+        case 307:
+        case 308:
+            return true;
+        default:
+            return false;
+        }
     }
 
     /* @Override */

src/main/java/com/ning/http/client/providers/netty/NettyResponse.java

@@ -170,7 +170,16 @@ public FluentCaseInsensitiveStringsMap getHeaders() {
     /* @Override */
 
     public boolean isRedirected() {
-        return (status.getStatusCode() >= 300) && (status.getStatusCode() <= 399);
+        switch (status.getStatusCode()) {
+        case 301:
+        case 302:
+        case 303:
+        case 307:
+        case 308:
+            return true;
+        default:
+            return false;
+        }
     }
 
     /* @Override */