Re: RFR: 8394607: HttpClient HTTP/2: fully received response body discarded when peer closes connection before InputStream is read (regression from JDK-8317808)

[Daniel Jeliński]

On Mon, 19 Oct 2026 17:47:27 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:

>> As long as we haven't processed all the buffered data? This is HTTP/2 so it 
>> is impossible to know whether you have received all the data before 
>> processing it.
>
> ok, what's the point of the errorref check then? Is it just to avoid the 
> costly getBody call?

Suggestion:

                    if (bodyCf == null || bodyCf.isCompletedExceptionally()) 
break;

isCancelled implies isCompletedExceptionally, no need to check both.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33385#discussion_r4317001395