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