RFR: 8377796: java.net.http.HttpClient.send() and sendAsync() never complete when BodyHandler.apply() returns null [v2]

Jaikiran Pai jpai at openjdk.org
Thu Feb 12 16:16:50 UTC 2026


> Can I please get a review of this change which proposes to address the issue noted in https://bugs.openjdk.org/browse/JDK-8377796?
> 
> The change here asserts that if `BodyHandler.apply()` returns `null`, then the HTTP request completes exceptionally. A new test has been introduced to reproduce the issue and verify the fix.

Jaikiran Pai has updated the pull request incrementally with three additional commits since the last revision:

 - split the test method arguments into individual parts
 - no need for the duplicate null check
 - use HttpTestEchoHandler in the new test

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/29691/files
  - new: https://git.openjdk.org/jdk/pull/29691/files/792461b7..52cd9049

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

  Stats: 52 lines in 3 files changed: 3 ins; 24 del; 25 mod
  Patch: https://git.openjdk.org/jdk/pull/29691.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/29691/head:pull/29691

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


More information about the net-dev mailing list