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