RFR: 8241372: Several test failures due to javax.net.ssl.SSLException: Connection reset [v2]
Rajan Halade
rhalade at openjdk.java.net
Fri Feb 5 17:49:44 UTC 2021
On Thu, 4 Feb 2021 14:55:39 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Fernando Guallini has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Merge branch '8241372' of github.com:fguallini/jdk into 8241372
>> - remove not needed bug id from tests, run with preferIPv4Stack
>
> test/jdk/javax/net/ssl/SSLSession/TestEnabledProtocols.java line 145:
>
>> 143: // The server side may have closed the socket.
>> 144: System.out.println("Client SSLException:");
>> 145: ssle.printStackTrace(System.out);
>
> If security / TLS experts agree that this is OK, then it is OK for me. Not being an expert, I'd be concerned that catching plain SSLException might be too wide and that this might hide errors. In that case, and if the error is intermittent, one possibility could be to respin the test in case of SSLException (do client side and server side again) and fail if the second attempts fails too.
You can use getCause() to examine reason for SSLException.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2405
More information about the security-dev
mailing list