On Tue, 13 May 2025 08:15:34 GMT, Volkan Yazici <vyaz...@openjdk.org> wrote:
> 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 Very nice! It would be good to add the `(streamId=<streamId>)` at line 676 too in Stream.java (Trailing Headers), and to add the whole line (`GET <uri> <status-code>`) at line 1782 (push response headers). ------------- PR Comment: https://git.openjdk.org/jdk/pull/25201#issuecomment-2876622516
