On Fri, 18 Aug 2023 13:30:54 GMT, Weibing Xiao <d...@openjdk.org> 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:
> 
>   re-write close method

1) Refactor the code again, and simplify the code to handle the setting of 
connection time
2) Add more test cases
3) Bring back the pre-created keystore. I could not find a good example to 
create the key store at runtime.

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

PR Comment: https://git.openjdk.org/jdk/pull/15294#issuecomment-1684688234

Reply via email to