RFR: 8328919: Add BodyHandlers / BodySubscribers methods to handle excessive server input [v7]

Michael McMahon michaelm at openjdk.org
Thu Jan 16 09:48:40 UTC 2025


On Thu, 16 Jan 2025 08:56:57 GMT, Volkan Yazıcı <duke at openjdk.org> wrote:

>> Adds `limiting()` factory methods to `HttpResponse.Body{Handlers,Subscribers}` to handle excessive server input in `HttpClient`. I would appreciate your input whether `discardExcess` should be kept or dropped. I plan to file a CSR once there is an agreement on the PR.
>
> Volkan Yazıcı has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Remove redundant import
>  - Remove redundant constant

src/java.net.http/share/classes/java/net/http/HttpResponse.java line 757:

> 755:          * consumed and passed to the given downstream {@code BodyHandler}}
> 756:          * <p>
> 757:          * If the number of body bytes received exceeds the the given

Suggestion:

         * If the number of body bytes received exceeds the given

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23096#discussion_r1918142563


More information about the net-dev mailing list