On Fri, 4 Jul 2025 08:34:52 GMT, Daniel Fuchs <dfu...@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
