On Mon, 9 Oct 2023 13:29:39 GMT, Daniel Fuchs <[email protected]> wrote:
>> Conor Cleary has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains ten additional
>> commits since the last revision:
>>
>> - 8309118: Fixed error in handleReset ReentrantLock
>> - 8309118: Updates to Stream cancelImpl
>> - 8309118: Refactored test, updated incoming_reset
>> - Merge branch 'master' into JDK-8309118
>> - 8309118: Cleanup identifiers and whitespace
>> - 8309118: Removed unused try-with-resources
>> - 8309118: Improve comments and annotations
>> - 8309118: Remove local test timeout value
>> - 8309118: Add more tests for 100 ExpectContinue with HTTP/2
>
> src/java.net.http/share/classes/jdk/internal/net/http/Stream.java line 195:
>
>> 193: if (endStreamReceived()) {
>> 194: // If END_STREAM is already received, we should
>> not receive any new RST_STREAM frames and
>> 195: // close the connection gracefully by
>> processing all remaining frames in the inputQ.
>
> Suggestion:
>
> // If END_STREAM is already received, we should we
> should simply
> // complete the requestBodyCF successfuly and stop
> sending any
> // request data.
How about just
// If END_STREAM is already received, complete the requestBodyCF successfully
// and stop sending any request data.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15664#discussion_r1364025064
