On Thu, 8 Dec 2022 18:28:43 GMT, Daniel Fuchs <[email protected]> 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 <[email protected]> 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
