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

Michael McMahon michaelm at openjdk.org
Fri Jul 4 13:10:40 UTC 2025


On Fri, 4 Jul 2025 08:34:52 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> The PlainHttpConnection::connectAsync method implements a retry logic that will call connect() again if connect() throws the first time. This will not work, as the channel is closed when connect() throws. That logic should be removed. Reconnection should only be attempted at a higher level (MultiExchange).
>
> Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review feedback

I think this code was added to deal with https://bugs.openjdk.org/browse/JDK-8256459. Did the change not fix that problem?

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

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


More information about the net-dev mailing list