Skip to content

Commit d9a218e

Browse files
author
Stephane Landelle
committed
Don't compute regular host if there's a virtual one
1 parent 9b157d3 commit d9a218e

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

providers/netty/src/main/java/org/asynchttpclient/providers/netty/NettyAsyncHttpProvider.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -577,12 +577,14 @@ private static SpnegoEngine getSpnegoEngine() {
577577

578578
private static HttpRequest construct(AsyncHttpClientConfig config, Request request, HttpMethod m, URI uri, ChannelBuffer buffer, ProxyServer proxyServer) throws IOException {
579579

580-
String host = AsyncHttpProviderUtils.getHost(uri);
580+
String host = null;
581581
boolean webSocket = isWebSocket(uri);
582582

583583
if (request.getVirtualHost() != null) {
584584
host = request.getVirtualHost();
585-
}
585+
} else {
586+
AsyncHttpProviderUtils.getHost(uri);
587+
}
586588

587589
HttpRequest nettyRequest;
588590
if (m.equals(HttpMethod.CONNECT)) {
@@ -607,9 +609,7 @@ else if (uri.getRawQuery() != null)
607609
}
608610

609611
if (host != null) {
610-
if (uri.getPort() == -1) {
611-
nettyRequest.setHeader(HttpHeaders.Names.HOST, host);
612-
} else if (request.getVirtualHost() != null) {
612+
if (request.getVirtualHost() != null || uri.getPort() == -1) {
613613
nettyRequest.setHeader(HttpHeaders.Names.HOST, host);
614614
} else {
615615
nettyRequest.setHeader(HttpHeaders.Names.HOST, host + ":" + uri.getPort());

0 commit comments

Comments
 (0)