On Wed, 16 Aug 2023 23:11:11 GMT, Weibing Xiao <[email protected]> wrote:
>> Please refer to JDK-8314063.
>>
>> The failure scenario is due to the setting of connection timeout. It is
>> either too small or not an optimal value for the system. When the client
>> tries to connect to the server with LDAPs protocol. It requires the
>> handshake after the socket is created and connected, but it fails due to
>> connection timeout and leaves the socket open. It is not closed properly due
>> to the exception handling in the JDK code.
>>
>> The change is adding a try/catch block and closing the socket in the catch
>> block, and the format of the code got changed consequently.
>
> Weibing Xiao has updated the pull request incrementally with one additional
> commit since the last revision:
>
> updated the code according to the review
src/java.naming/share/classes/com/sun/jndi/ldap/Connection.java line 287:
> 285: // create the socket with factory
> 286: if (socketFactory != null) {
> 287: socket = createSocketWithFactory (host, port,
> socketFactory, connectTimeout) ;
you could use method overloading :-)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15294#discussion_r1296540658
