Commiteace5fe

oleksiys

committed

[1.8.x] + don't add another Host header if it's already set

1 parentae87158 commiteace5feCopy full SHA for eace5fe

File tree

+10

-7

lines changed

src/main/java/com/ning/http/client/providers/grizzly
- GrizzlyAsyncHttpProvider.java

+10

-7

lines changed

Lines changed: 10 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -873,14 +873,17 @@ private boolean sendAsGrizzlyRequest(final Request request,`
`873`	`873`	`booleansecure ="https".equals(uri.getScheme());`
`874`	`874`	`builder.method(method);`
`875`	`875`	`builder.protocol(Protocol.HTTP_1_1);`
`876`		`-Stringhost =request.getVirtualHost();`
`877`		`-if (host !=null) {`
`878`		`-builder.header(Header.Host,host);`
`879`		`- }else {`
`880`		`-if (uri.getPort() == -1) {`
`881`		`-builder.header(Header.Host,uri.getHost());`
	`876`	`+`
	`877`	`+if (!request.getHeaders().containsKey(Header.Host.toString())) {`
	`878`	`+Stringhost =request.getVirtualHost();`
	`879`	`+if (host !=null) {`
	`880`	`+builder.header(Header.Host,host);`
`882`	`881`	`}else {`
`883`		`-builder.header(Header.Host,uri.getHost() +':' +uri.getPort());`
	`882`	`+if (uri.getPort() == -1) {`
	`883`	`+builder.header(Header.Host,uri.getHost());`
	`884`	`+ }else {`
	`885`	`+builder.header(Header.Host,uri.getHost() +':' +uri.getPort());`
	`886`	`+ }`
`884`	`887`	`}`
`885`	`888`	`}`
`886`	`889`	`finalProxyServerproxy =ProxyUtils.getProxyServer(config,request);`

Comments

(0)