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

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


> ### **Description**
> When using `HttpURLConnection::setChunkedStreamingMode()`, the current documentation does not make it clear that the `chunklen` parameter refers to the amount of bytes sent on the wire per HTTP/1.1 chunk and _not_ the  payload length per chunk. The `chunklen` parameter takes into account the bytes used for the chunk size header, two CRLF characters and the possible payload length inclusively.
> 
> ### **Summary of Changes & Justification**
> The API text has been updated to reflect the actual behavior of the method as described above to avoid a similar bug being submitted in the future and to make it clear what the `chunklen` parameter specifies.
> 
> This change will require a csr.

Conor Cleary has updated the pull request incrementally with one additional commit since the last revision:

  8287134: Shortened text and corrected typo

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/12137/files
  - new: https://git.openjdk.org/jdk/pull/12137/files/93588aec..38de2297

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=12137&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12137&range=00-01

  Stats: 5 lines in 1 file changed: 0 ins; 0 del; 5 mod
  Patch: https://git.openjdk.org/jdk/pull/12137.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12137/head:pull/12137

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


More information about the net-dev mailing list