RFR: 8361249: PlainHttpConnection connection logic can be simplified [v3]

[Daniel Fuchs]

On Fri, 4 Jul 2025 13:08:28 GMT, Michael McMahon <michaelm at openjdk.org> wrote:

>I think this code was added to deal with https://bugs.openjdk.org/browse/JDK-8256459. Did the change not fix that problem? Or did that change include the changes to MultiExchange you mention above?

The change did not include changes to the multi exchange but it included changes to the tests themselves.
I fail to see how calling connect() again on the same channel could have solved the issue: I verified by code inspection that the channel is closed when connect() fails - and calling connect() again simply generates a ClosedChannelException.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/26087#issuecomment-3036323261