On Fri, 19 Apr 2024 15:47:15 GMT, robert engels <d...@openjdk.org> wrote:
>> test/jdk/com/sun/net/httpserver/bugs/B6968351.java line 90:
>>
>>> 88: is.close();
>>> 89: rmap.add("content-type","text/plain");
>>> 90: t.sendResponseHeaders(200,5);
>>
>> if I read the code correctly, there might be a similar delay when sending
>> the last chunk of a chunked response. Would you like to fix it here as well?
>> If not, we can file another ticket for it.
>>
>> In order to send a chunked response, change the second parameter to `0` here.
>
> If you send a chunked response you must close the exchange, or close the
> output stream, or there is no way to determine the data is done.
The problem with closing a chunked stream is that it flushes both before and
after the final chunk. I assume it may also cause delays.
The only case when you don't need to close the exchange or the output stream is
when your response contains no content.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18667#discussion_r1572602425
