RFR: 6968351: httpserver clashes with delayed TCP ACKs for low Content-Length [v6]

[Daniel Fuchs]

On Fri, 19 Apr 2024 18:49:33 GMT, robert engels <duke at openjdk.org> wrote:

>> src/jdk.httpserver/share/classes/sun/net/httpserver/ChunkedOutputStream.java line 142:
>> 
>>> 140:         try {
>>> 141:             /* write any pending chunk data */
>>> 142:             writeChunk();
>> 
>> Isn't there a risk that you might write two empty chunks here if there are no pending chunks?
>
> yes, let me fixed that.

Shouldn't you look at whether count > 0 as is  done in flush() ?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18667#discussion_r1572782506