RFR: 8263442: Potential bug in jdk.internal.net.http.common.Utils.CONTEXT_RESTRICTED [v3]

Daniel Fuchs dfuchs at openjdk.java.net
Tue Mar 23 11:09:41 UTC 2021


On Mon, 22 Mar 2021 22:16:09 GMT, Michael McMahon <michaelm at openjdk.org> wrote:

>> Hi,
>> 
>> The fix for the reported bug in Utils.CONTEXT_RESTRICTED caused a couple of regression failures, which turned out to be another bug exposed by this fix where HTTP/1.1 CONNECT requests with authentication were filtering out proxy authentication headers wrongly. This was because the HttpRequestImpl created for the repeated CONNECT was putting the system headers in the user headers area of the HttpRequestImpl. The fix for that is to supply the user and system headers direct to the place where the new HttpRequestImpl is created.
>> 
>> Thanks
>> Michael
>
> Michael McMahon has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Updated test to test proxy authorization

Marked as reviewed by dfuchs (Reviewer).

test/jdk/java/net/httpclient/AuthFilter.java line 145:

> 143:         if (useProxy) {
> 144:             proxy = new ProxyServer();
> 145:             proxyAddr = new InetSocketAddress("127.0.0.1", proxy.getPort());

Maybe the test should be guarded in case the machine is IPv6 only

test/jdk/java/net/httpclient/AuthFilter.java line 160:

> 158:         if (useProxy) {
> 159:             builder.proxy(ProxySelector.of(proxyAddr));
> 160:         }

You should probably set NO_PROXY otherwise to avoid the default proxy selector on mac.

-------------

PR: https://git.openjdk.java.net/jdk/pull/2977


More information about the net-dev mailing list