Merge pull request AsyncHttpClient#137 from levinotik/netty-lowercase-redirect-fix

jfarcand · jfarcand · commit 06318999148f · 2012-09-06T04:25:04.000-07:00
FIX ISSUE AsyncHttpClient#127 NettyAsyncHttpProdiver redirects ignoring case
diff --git a/providers/netty/src/main/java/com/ning/http/client/providers/netty/NettyAsyncHttpProvider.java b/providers/netty/src/main/java/com/ning/http/client/providers/netty/NettyAsyncHttpProvider.java
@@ -1988,7 +1988,7 @@ private boolean redirect(Request request,
                 String location = response.getHeader(HttpHeaders.Names.LOCATION);
                 URI uri = AsyncHttpProviderUtils.getRedirectUri(future.getURI(), location);
                 boolean stripQueryString = config.isRemoveQueryParamOnRedirect();
-                if (!uri.toString().equalsIgnoreCase(future.getURI().toString())) {
+                if (!uri.toString().equals(future.getURI().toString())) {
                     final RequestBuilder nBuilder = stripQueryString ?
                             new RequestBuilder(future.getRequest()).setQueryParameters(null)
                             : new RequestBuilder(future.getRequest());
