On Tue, 21 Jan 2025 15:03:54 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> test/jdk/java/net/httpclient/HttpResponseLimitingTest.java line 164:
>> 
>>> 162:         HttpClient.Builder builder = HttpClient
>>> 163:                 .newBuilder()
>>> 164:                 .connectTimeout(Duration.ofSeconds(5));
>> 
>> Same comment about the usage of a timeout.
>
> I'd suggest to pass `.proxy(NO_PROXY)` too.

Removed the timeouts and added `NO_PROXY` in 
e10771317ef57f80d25be58df77eb5c5b0083b0c.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23096#discussion_r1924296251

Reply via email to