> 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
