RFR: 8266900: java/net/httpclient/ShortResponseBody.java fails on windows with java.io.IOException: Unable to establish loopback connection [v2]
Daniel Jeliński
djelinski at openjdk.org
Thu Oct 20 17:22:55 UTC 2022
On Thu, 20 Oct 2022 13:49:42 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Really change client every 10 requests
>
> test/jdk/java/net/httpclient/ShortResponseBody.java line 135:
>
>> 133: }
>> 134: }
>> 135: numberOfRequests++;
>
> Did you mean `numberOfRequests = (numberOfRequests % REQUESTS_PER_CLIENT) + 1;` to match the description of the changes?
Good catch! Forgot to reset the counter. Fix pushed.
> test/jdk/java/net/httpclient/ShortResponseBody.java line 706:
>
>> 704: @AfterTest
>> 705: public void teardown() throws Exception {
>> 706: if (sharedClient != null) sharedClient = null;
>
> should probably null `client` here?
Not necessary. `client` is an instance field and will be GCed as soon as the framework is done with the test.
-------------
PR: https://git.openjdk.org/jdk/pull/10794
More information about the net-dev
mailing list