RFR: 8343791: Socket.connect API should document whether the socket will be closed when hostname resolution fails or another error occurs [v27]

Jaikiran Pai jpai at openjdk.org
Tue Dec 3 07:01:51 UTC 2024


On Mon, 2 Dec 2024 08:12:32 GMT, Volkan Yazıcı <duke at openjdk.org> wrote:

>> This PR, addressing 8343791, changes `Socket::connect()` methods to close the `Socket` in the event that the connection cannot be established, the timeout expires before the connection is established, or the socket address is unresolved.
>> 
>> `tier3` tests pass against the 9f00f61d3b7fa42a5e23a04f80bb4bb1a2076ef2.
>
> Volkan Yazıcı has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Reword the `ConnectFailTest` summary

I see that Alan and Daniel have approved the latest state of this PR and the CSR has been approved too. tier1, tier2 and tier3 testing has been reported as successful too. I'll go ahead and sponsor this change now.

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

PR Comment: https://git.openjdk.org/jdk/pull/22160#issuecomment-2513706605


More information about the net-dev mailing list