Hi Robert, I am waiting for the OCA to be processed before looking at your PR.
FWIW some of the recent tests we have added for the HttpServer already use the HttpClient - so unless your test/fix does require HttpURLConnection it is fine to use HttpClient. Were you planning to later propose to backport this change back to JDK 8? best regards, -- daniel On 08/04/2024 14:23, Robert Engels wrote:
I’ll update my PR to remove the nodelay setting from those tests. I am going to change the PR to use UrlConnection rather than HttpClient as well.