[jdk20] RFR: 8298340: java/net/httpclient/CancelRequestTest.java fails with AssertionError: Found some subscribers for testPostInterrupt [v2]

Daniel Fuchs dfuchs at openjdk.org
Fri Dec 9 13:57:00 UTC 2022


> There are several ways by which an HTTP/2 stream can be closed. Due to the built-in asynchronous behavior, and to avoid unnecessary churn when a request is cancelled/aborted there are two places where the boolean stream's state closed can be set to true: cancelImpl and close. The current code completes the subscriber in cancelImpl, but the subscriber should also be completed in close. The intermittent failure happens if close gets ever called before cancelImpl.
> This patch fixes that.

Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:

  Integrated review feedback

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

Changes:
  - all: https://git.openjdk.org/jdk20/pull/3/files
  - new: https://git.openjdk.org/jdk20/pull/3/files/eb998838..2a0e36ce

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

  Stats: 24 lines in 1 file changed: 6 ins; 17 del; 1 mod
  Patch: https://git.openjdk.org/jdk20/pull/3.diff
  Fetch: git fetch https://git.openjdk.org/jdk20 pull/3/head:pull/3

PR: https://git.openjdk.org/jdk20/pull/3


More information about the net-dev mailing list