Skip to content

Commit c0f338c

Browse files
committed
Removed, what appeared to be, a reduntant block of code. The builder is already configured using the existing request.
If this block executes, it pushes the same url that the request already has. Removing the block resolves issue AsyncHttpClient#62. Existing tests continue to pass. Reviewed by JFA.
1 parent fda1596 commit c0f338c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1998,9 +1998,9 @@ public void handle(final ChannelHandlerContext ctx, final MessageEvent e) throws
19981998
final FluentCaseInsensitiveStringsMap headers = request.getHeaders();
19991999
final RequestBuilder builder = new RequestBuilder(future.getRequest());
20002000

2001-
if (realm != null && !future.getURI().getPath().equalsIgnoreCase(realm.getUri())) {
2002-
builder.setUrl(future.getURI().toString());
2003-
}
2001+
//if (realm != null && !future.getURI().getPath().equalsIgnoreCase(realm.getUri())) {
2002+
// builder.setUrl(future.getURI().toString());
2003+
//}
20042004

20052005
if (statusCode == 401
20062006
&& wwwAuth.size() > 0

0 commit comments

Comments
 (0)