RFR: 8351347: HttpClient Improve logging of response headers [v2]

Volkan Yazici vyazici at openjdk.org
Wed May 14 09:31:07 UTC 2025


> Includes request method, request URI, response status code, and HTTP/2 stream ID while logging response headers in the HTTP Client.
> 
> ### Demonstration
> 
> Snippets from running JTreg against `test/jdk/java/net/httpclient/HeadTest.java`:
> 
> **Before:**
> 
> INFO: HEADERS: RESPONSE HEADERS:
>     connection: Upgrade
>     upgrade: h2c
> ...
> INFO: HEADERS: RESPONSE HEADERS:
>     :status: 304
>     content-length: 300
> 
> 
> **After:** 
> 
> INFO: HEADERS: RESPONSE HEADERS:
>   GET http://127.0.0.1:48497/ 101
>     connection: Upgrade
>     upgrade: h2c
> ...
> INFO: HEADERS: RESPONSE HEADERS (streamid=1):
>   GET http://127.0.0.1:48497/ 304
>     :status: 304
>     content-length: 300

Volkan Yazici has updated the pull request incrementally with three additional commits since the last revision:

 - Match header logging in `Http1Request` with others
 - Shorten code
 - Improve header logging in more places

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25201/files
  - new: https://git.openjdk.org/jdk/pull/25201/files/dee90eba..be71467e

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25201&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25201&range=00-01

  Stats: 40 lines in 4 files changed: 8 ins; 12 del; 20 mod
  Patch: https://git.openjdk.org/jdk/pull/25201.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25201/head:pull/25201

PR: https://git.openjdk.org/jdk/pull/25201


More information about the net-dev mailing list