Skip to content

Commit 3df88ef

Browse files
committed
FIX ISSUE #127 NettyAsyncHttpProdiver redirects ignoring case
Issue #127 mentions that the NettyHttpProvider ignores case when checking for equality between the redirect Uri and the Future's Uri. This commit simply changes the equalsIgnoreCase to equals.
1 parent a96602a commit 3df88ef

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1987,7 +1987,7 @@ private boolean redirect(Request request,
19871987
String location = response.getHeader(HttpHeaders.Names.LOCATION);
19881988
URI uri = AsyncHttpProviderUtils.getRedirectUri(future.getURI(), location);
19891989
boolean stripQueryString = config.isRemoveQueryParamOnRedirect();
1990-
if (!uri.toString().equalsIgnoreCase(future.getURI().toString())) {
1990+
if (!uri.toString().equals(future.getURI().toString())) {
19911991
final RequestBuilder nBuilder = stripQueryString ?
19921992
new RequestBuilder(future.getRequest()).setQueryParameters(null)
19931993
: new RequestBuilder(future.getRequest());

0 commit comments

Comments
 (0)