On Mon, 15 Dec 2025 15:34:52 GMT, Matthias Baesken <[email protected]> wrote:
>> Arno Zeller has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains four additional
>> commits since the last revision:
>>
>> - Use CountDownLatch
>> - Merge remote-tracking branch 'upstream/master' into JDK-8371559
>> - Merge branch 'openjdk:master' into JDK-8371559
>> - Use timeout factor when waiting for server ready
>
> test/jdk/javax/net/ssl/Stapling/HttpsUrlConnClient.java line 475:
>
>> 473: * Release the client, if not active already...
>> 474: */
>> 475: System.err.println("Server died: " + e);
>
> Maybe do e.printStackTrace(System.err); too ? but leave it to you , it is
> now at least better than before ....
> (same for some other exception handling coding you changed)
For this change I will leave it as it is
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28784#discussion_r2626155808
