On Thu, 8 Dec 2022 15:20:47 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.
>
> Daniel Fuchs has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Fix typo

Discussion continues there: https://github.com/openjdk/jdk20/pull/3

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

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

Reply via email to