On Fri, 19 Apr 2024 16:17:35 GMT, Michael McMahon <micha...@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

Reply via email to