On Thu, 9 Oct 2025 04:02:41 GMT, Josiah Noel <[email protected]> wrote:

>> Following the guideline of the last comment on 
>> [JDK-8349670](https://bugs.openjdk.org/browse/JDK-8349670?focusedId=14794649&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14794649),
>>  resolves the issue where sending a 1xx status code would close the input 
>> stream, preventing the server from reading the body. 
>> 
>> - When a 1xx status code is sent by `sendResponseHeaders`, the input/output 
>> streams will not be closed prematurely.
>> - sentHeaders will not be set to true when sending 1xx status codes
>> - 100-continue will be sent automatically when trying to read the 
>> inputstream if `Expect: 100-continue` header is present
>
> Josiah Noel has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   allow manual 100-Continue processing

Hi guys, I would prefer if we handled any changes to the 100 - continue 
behaviour in a separate fix. That would be easier to review and to discuss 
options. So I'd suggest to revert 
[4b00588](https://github.com/openjdk/jdk/pull/27069/commits/4b0058886cf10b1703426acc4131453c46636645)
 in this PR and possibly discuss that in a followup. Lets keep separate issues 
separate. The code that handles 100 in ServerImpl is attributed to `duke` which 
tells me it's been there for a long time :-)

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

PR Comment: https://git.openjdk.org/jdk/pull/27069#issuecomment-3384961130

Reply via email to