On Thu, 6 Feb 2025 15:16:30 GMT, Matthew Donovan <mdono...@openjdk.org> wrote:
>> Mikhail Yankelevich has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Apply suggestions from cr >> >> Co-authored-by: Daniel Fuchs <67001856+df...@users.noreply.github.com> > > test/jdk/sun/net/www/protocol/https/HttpsURLConnection/CloseKeepAliveCached.java > line 225: > >> 223: System.setProperty("javax.net.ssl.trustStorePassword", passwd); >> 224: >> 225: System.setProperty("javax.net.debug", "all"); > > Is this printing a lot of stuff to the logs? Why did you remove the `if > (debug)`? Parsing it below, so now its necessary ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23469#discussion_r1944945331