On Fri, 19 Apr 2024 15:41:54 GMT, robert engels <d...@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. ------------- PR Comment: https://git.openjdk.org/jdk/pull/18667#issuecomment-2066892828
