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

[Daniel Fuchs]

On Thu, 8 Dec 2022 18:28:43 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 now been integrated.

Changeset: 05d67f69
Author:    Daniel Fuchs <dfuchs at openjdk.org>
URL:       https://git.openjdk.org/jdk20/commit/05d67f69e34a76702406b36436ddb5db18e8fa68
Stats:     51 lines in 4 files changed: 46 ins; 1 del; 4 mod

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

Reviewed-by: jpai

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

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