On Tue, 16 Dec 2025 09:05:49 GMT, Daniel Fuchs <[email protected]> wrote:
>> Volkan Yazici has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Move `SSLContext::setDefault` to static blocks > > test/jdk/com/sun/net/httpserver/simpleserver/HttpsServerAlertTest.java line > 72: > >> 70: LOGGER.addHandler(ch); >> 71: } >> 72: SSLContext.setDefault(sslContext); > > You could move that to a static block after line 62 I guess. Changed as requested in c7ef9bd5. > test/jdk/com/sun/net/httpserver/simpleserver/HttpsServerTest.java line 80: > >> 78: LOGGER.addHandler(ch); >> 79: } >> 80: SSLContext.setDefault(sslContext); > > Same here after line 70 Changed as requested in c7ef9bd5. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/28800#discussion_r2622632973 PR Review Comment: https://git.openjdk.org/jdk/pull/28800#discussion_r2622633246
