RFR: 8249824: s/n/w/p/https/HttpsURLConnection/CloseKeepAliveCached.java uses @ignore w/o bugid [v2]

Mikhail Yankelevich duke at openjdk.org
Thu Feb 6 15:39:12 UTC 2025


On Thu, 6 Feb 2025 15:16:30 GMT, Matthew Donovan <mdonovan at 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+dfuch at 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


More information about the net-dev mailing list