Withdrawn: 8298340: java/net/httpclient/CancelRequestTest.java fails with AssertionError: Found some subscribers for testPostInterrupt

Daniel Fuchs dfuchs at openjdk.org
Thu Dec 8 18:28:41 UTC 2022


On Thu, 8 Dec 2022 15:02:34 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

> 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.

This pull request has been closed without being integrated.

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

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


More information about the net-dev mailing list