RFR: 8293786: HttpClient will not send more than 64 kb of data from the 2nd request in http2 [v6]

Conor Cleary ccleary at openjdk.org
Thu May 4 15:46:18 UTC 2023


On Thu, 4 May 2023 15:24:06 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

> The proposed updates look reasonable. I wonder if we could somehow move this code to a method that could be called from both places.
> 
> For instance something like:
> 
> ```
> boolean sendResetIfUnconsumed(BodyInputStream bis) {
>     if ((!bis.isEof() || bis.q.size() > 0)) {
>         conn.outputQ.put(new ResetFrame(streamid, ResetFrame.NO_ERROR));
>     }
> } 
> ```

Maybe that kind of method you described could be placed in BodyInputStream, then it could be called from the Exchange and BodyOutputStream easily.

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

PR Comment: https://git.openjdk.org/jdk/pull/12694#issuecomment-1535006537


More information about the net-dev mailing list