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

Jaikiran Pai jpai at openjdk.org
Tue Jan 21 10:41:38 UTC 2025


On Mon, 20 Jan 2025 08:31:18 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 one additional commit since the last revision:
> 
>   Fix `HttpResponse` copyright year

test/jdk/java/net/httpclient/HttpResponseLimitingTest.java line 8:

> 6:  * under the terms of the GNU General Public License version 2 only, as
> 7:  * published by the Free Software Foundation.  Oracle designates this
> 8:  * particular file as subject to the "Classpath" exception as provided

Hello Volkan, for the test files, we don't use the "Classpath" exception copyright header. You can copy over the copyright text from another existing test file, to get the correct header.

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

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


More information about the net-dev mailing list