RFR: 8340182: Java HttpClient does not follow default retry limit of 3 retries

[Volkan Yazici]

On Thu, 29 May 2025 16:03:11 GMT, p-nima <duke at openjdk.org> wrote:

>> test/jdk/java/net/httpclient/HttpClientRetryLimitTest.java line 66:
>> 
>>> 64:             };
>>> 65: 
>>> 66:             httpTestServer.addHandler(httpTestHandler, "/");
>> 
>> In the past, there has been occasions in CI where a test server received connections from a totally unrelated test running in parallel on the same host. To avoid such mishaps, we better salt the path a bit by, say, appending the class' simple name?
>
> Thanks for the catch - the class simple name has been appended in 18bac9fa64f81110c2894f5f141e88ec5dc20b03

@p-nima, the handler still accepts all calls to `/`, you only salted the request URI. Would you mind doing the same in `httpTestServer.addHandler(...)` too, please? (You better create a `String requestPath` variable and use it both at the handler and the request.)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25490#discussion_r2115448674