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