RFR: 8287134: HttpURLConnection chunked streaming mode doesn't enforce specified size [v2]

Conor Cleary ccleary at openjdk.org
Mon Jan 23 12:29:20 UTC 2023


On Mon, 23 Jan 2023 11:40:35 GMT, Michael McMahon <michaelm at openjdk.org> wrote:

>> Conor Cleary has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8287134: Shortened text and corrected typo
>
> src/java.base/share/classes/java/net/HttpURLConnection.java line 285:
> 
>> 283:      *          and a minimum payload length of 1 byte. If chunklen is less
>> 284:      *          than or equal to 5, a default value will be used.
>> 285:      *
> 
> I'd suggest something like this:
> 
> chunklen The number of bytes to be written in
> each chunk, including the chunk header as a
> hexadecimal string (minimum of 1 byte), two CRLF's (4 bytes)
> and a minimum payload length of 1 byte. If chunklen is less
> than or equal to 5, a higher default value will be used.

Thanks @Michael-Mc-Mahon, I updated the text with respect to your suggestion there (which was definitely a bit more cleared than mine) and corrected my incorrect spelling of hexadecimal in my most recent changes just now.

-------------

PR: https://git.openjdk.org/jdk/pull/12137


More information about the net-dev mailing list