On Thu, 16 Nov 2023 16:32:14 GMT, Conor Cleary <[email protected]> wrote:

>> test/jdk/java/net/httpclient/RedirectTimeoutTest.java line 63:
>> 
>>> 61: import static java.net.http.HttpClient.Version.HTTP_1_1;
>>> 62: import static java.net.http.HttpClient.Version.HTTP_2;
>>> 63: 
>> 
>> Unless the problem affects HTTP/2 we probably should take the HTTP/2 aspect 
>> of the test out ..
>
> I did the testing with HTTP/2 as well because the bug was at the 
> MultiExchange level. Though the issue didn't seem to appear when using HTTP/2

So taking it out is probably ok 👍

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16689#discussion_r1396008495

Reply via email to