RFR: 8371471: HttpClient: Log HTTP/3 handshake failures if logging errors is enabled [v4]

Daniel Jeliński djelinski at openjdk.org
Sat Nov 8 20:24:01 UTC 2025


On Fri, 7 Nov 2025 17:40:22 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Surprisingly if the QUIC handshake fails exceptionally, and `-Djdk.httpclient.HttpClient.log=errors` is specified, the exception is not logged. This change fixes that.
>
> Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review feedback - request should fail

Doing this in a separate cleanup is fine by me. Note that one of the duplicates can be easily removed already.

src/java.net.http/share/classes/jdk/internal/net/http/common/Log.java line 394:

> 392:     public static void logError(Throwable t) {
> 393:         if (errors()) {
> 394:             logger.log(Level.INFO, "ERROR: " + t, t);

Suggestion:

            logger.log(Level.INFO, "ERROR:", t);

-------------

Marked as reviewed by djelinski (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/28196#pullrequestreview-3438579143
PR Review Comment: https://git.openjdk.org/jdk/pull/28196#discussion_r2507114008


More information about the net-dev mailing list