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

Reply via email to