NettyAsyncHttpProvider does not work work with -Dhttp.nonProxyHosts

[martinei]

If a Proxy and an exception are vonfigured via http.proxyHost and http.nonProxyHost, the NettyAsyncHttpProvider will use the complete URL as Path (Which usually leads to a 404).

An Request that was meant to be issued against http://localhost:8080/ will look like this:

GET http://localhost:8080/ HTTP/1.1
Host: localhost:8080
...

The Problem is in NettyAsyncHttpProvider, line 572 and following:

if (isProxyServer(config, request))
                path = new StringBuilder(uri.toString());
            else {
                path = new StringBuilder(uri.getRawPath());
                if (uri.getQuery() != null) {
                    path.append("?").append(uri.getRawQuery());
                }
            }

isProxyServer returns true, even if the host is configured in nonProxyHost. It should better check ProxyUtils.avoidProxy(proxyServer, request);

[w31rd0]

Same thing (complete URL being used instead of path) happens when the request is https and uses a proxy.

In that case first a CONNECT request is made via the proxy, and then a regular request should be made over the established secure connection, but isProxyServer(config, request) condition leads to the complete URL being used as a path, and that causes 404 error.

[slandelle]

@w31rd0 This is a different issue, could you open a new one and elaborate, please?