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