On Thu, 9 Oct 2025 14:53:22 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:
> 
>   Revert "Update InputRead100Test.java"

I'd suggesst to rename `InputRead100Test` to something like 
`Send1xxResponsesTest` and add more methods to it.
Rename the current methods in that test to include 100 in their name, since 
they are testing the special case of 100.
Add an additional test method where the server sends code `123` 3 times before 
sending the final response and check that the client receives all three.

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

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

Reply via email to