On Thu, 28 Nov 2024 16:20:58 GMT, Alan Bateman <[email protected]> wrote:
>>> I checked the socket adaptor and there is needed a bug here. SocketChannel
>>> throws AlreadyConnectedException and the socket adaptor should map that to
>>> SocketException("already connected"). It doesn't do that so we should fix
>>> it.
>>
>> @AlanBateman, I presume you want me to fix that mapping in a separate
>> JBS+PR, right?
>
> If Daniel is okay with it, then it might be easier to fix it here as part of
> this PR because the changes to add the mapping are touching exactly the focus
> of this PR, and the new test is testing the condition that exposes this.
@AlanBateman, I've updated `Net` and `ConnectFailTest`. Would you mind checking
them, please? (In the meantime, I will start a `tier1..3` run and update the
CSR.)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22160#discussion_r1863188789
