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

Volkan Yazıcı duke at openjdk.org
Tue Jan 14 20:04:40 UTC 2025


> 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 four additional commits since the last revision:

 - Make `LimitingSubscriber` stateful
 - Improve Javadoc
 - Replace `IllegalStateException` with an `IOException`
 - Remove `discardExcess` argument

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/23096/files
  - new: https://git.openjdk.org/jdk/pull/23096/files/3c9e4eae..a4fd1bab

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

  Stats: 146 lines in 3 files changed: 34 ins; 85 del; 27 mod
  Patch: https://git.openjdk.org/jdk/pull/23096.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/23096/head:pull/23096

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


More information about the net-dev mailing list