Integrated: 8361249: PlainHttpConnection connection logic can be simplified

Daniel Fuchs dfuchs at openjdk.org
Mon Jul 7 08:24:46 UTC 2025


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

This pull request has now been integrated.

Changeset: 8ad48368
Author:    Daniel Fuchs <dfuchs at 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


More information about the net-dev mailing list