RFR: 6968351: httpserver clashes with delayed TCP ACKs for low Content-Length [v2]
robert engels
duke at openjdk.org
Fri Apr 19 16:41:57 UTC 2024
On Fri, 19 Apr 2024 16:17:35 GMT, Michael McMahon <michaelm at openjdk.org> wrote:
> > This is not as complex as I expected it to be. Which is a good thing!
> > Some tests are failing with this change. See:
> > ```
> > test/jdk/sun/net/www/http/KeepAliveCache/B8293562.java
> > test/jdk/java/net/Authenticator/B4769350.java
> > ```
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> > These tests don't close the exchange or the output stream after sending the response, and the headers are never flushed. Users might have similar code, so I think a release note might be needed.
>
> This is a concern. Release note needed at least or I wonder if the new behavior should be enabled through a configuration switch. Probably not worth going to that extreme, but we need to consider the compatibility impact carefully.
We won't need a release note. I figured out how to fix it.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/18667#issuecomment-2066923739
More information about the net-dev
mailing list