On Thu, 11 Feb 2021 12:08:54 GMT, Fernando Guallini <fguall...@openjdk.org> 
wrote:

>> The server side is binding to the wildcard address which has been a source 
>> of instability in many networking tests due to javax.net.ssl.SSLException: 
>> Connection reset. Changing the following tests to bind to loopback address 
>> fixes intermittent failures:
>> sun/security/ssl/SSLSocketImpl/ReverseNameLookup.java
>> javax/net/ssl/TLSCommon/TLSTest.java
>> sun/security/ssl/CipherSuite/SupportedGroups.java
>> 
>> 
>> javax/net/ssl/SSLSession/TestEnabledProtocols.java still throws a connection 
>> reset occasionally because the server closes the connection before the 
>> client is done during handshake. That race condition cannot be completely 
>> removed in this test, thus is now handled and logged.
>
> Fernando Guallini has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   check exception type

test/jdk/javax/net/ssl/SSLSession/TestEnabledProtocols.java line 155:

> 153:                 && "Connection reset".equals(cause.getMessage())) {
> 154:             System.out.println("Client SSLException:");
> 155:             ssle.printStackTrace(System.out);

Sorry for nit picking - but I would keep that tracing in the catch clause 
above. 

        } catch (SSLException ssle) {
            if (isConnectionReset(ssle)) {
                System.out.println("Client SSLException:");
                ssle.printStackTrace(System.out);
            } else {
                 failTest(ssle, "Client got UNEXPECTED SSLException:");
            }
        }
 ```

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

PR: https://git.openjdk.java.net/jdk/pull/2405

Reply via email to