RFR: 8369595: HttpClient: HttpHeaders.firstValueAsLong failures should be converted to ProtocolException [v2]
Volkan Yazici
vyazici at openjdk.org
Tue Nov 25 21:10:46 UTC 2025
On Tue, 25 Nov 2025 12:57:02 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Volkan Yazici has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Address review remarks
>
> src/java.net.http/share/classes/jdk/internal/net/http/Http3ExchangeImpl.java line 1289:
>
>> 1287: // Read & validate `Content-Length`
>> 1288: long clen = readContentLength(
>> 1289: promiseHeaders, "illegal push headers for pushId=%s: ".formatted(pushId), 0);
>
> Let's keep -1 as default value to represent no Content-length header, even though not strictly necessary in this context.
Changed as requested in dab06dea528.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28431#discussion_r2561404387
More information about the net-dev
mailing list