On Tue, 25 Nov 2025 12:57:02 GMT, Daniel Fuchs <[email protected]> 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

Reply via email to