On Thu, 4 Feb 2021 14:55:39 GMT, Daniel Fuchs <dfu...@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