Skip to content

Commit fb2fac9

Browse files
committed
Properly reuse connection on redirect when Location port is the default one, close AsyncHttpClient#870
1 parent 6f46600 commit fb2fac9

File tree

1 file changed

+2
-2
lines changed
  • src/main/java/com/ning/http/client/providers/netty/handler

1 file changed

+2
-2
lines changed

src/main/java/com/ning/http/client/providers/netty/handler/Protocol.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515

1616
import static com.ning.http.client.providers.netty.util.HttpUtils.HTTP;
1717
import static com.ning.http.client.providers.netty.util.HttpUtils.WEBSOCKET;
18-
import static com.ning.http.util.AsyncHttpProviderUtils.followRedirect;
18+
import static com.ning.http.util.AsyncHttpProviderUtils.*;
1919
import static org.jboss.netty.handler.codec.http.HttpHeaders.Names.*;
2020
import static org.jboss.netty.handler.codec.http.HttpResponseStatus.*;
2121

@@ -160,7 +160,7 @@ protected boolean exitAfterHandlingRedirect(//
160160

161161
boolean redirectToSameHost = request.getUri().getScheme().equals(nextRequest.getUri().getScheme())
162162
&& request.getUri().getHost().equals(nextRequest.getUri().getHost())
163-
&& request.getUri().getPort() == nextRequest.getUri().getPort();
163+
&& getDefaultPort(request.getUri()) == getDefaultPort(nextRequest.getUri());
164164

165165
if (redirectToSameHost) {
166166
future.setReuseChannel(true);

0 commit comments

Comments
 (0)