On Wed, 2 Jul 2025 08:37:34 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).

This pull request has now been integrated.

Changeset: 8ad48368
Author:    Daniel Fuchs <dfu...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/8ad48368838588846324550c455c66ae86f1226b
Stats:     60 lines in 1 file changed: 2 ins; 52 del; 6 mod

8361249: PlainHttpConnection connection logic can be simplified

Reviewed-by: djelinski, vyazici, michaelm, jpai

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

PR: https://git.openjdk.org/jdk/pull/26087

Reply via email to