The OCA tag has been removed. So it’s ok to review now.
> On Apr 8, 2024, at 9:10 AM, Robert Engels <reng...@ix.netcom.com> wrote: > > And thanks for the note about HttpClient - ill leave that as is then. > >> On Apr 8, 2024, at 9:06 AM, Robert Engels <reng...@ix.netcom.com> wrote: >> >> It’s fairly isolated but I haven’t checked to see how the JDK differs in >> jdk8 - but I’m guessing not very much so it should be a trivial port. >> >>>> On Apr 8, 2024, at 9:03 AM, Daniel Fuchs <daniel.fu...@oracle.com> wrote: >>> >>> 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. >>> >>>
