RFR: 8349670: HttpServer: sending interim responses fails after JDK-7026262 [v21]
Josiah Noel
duke at openjdk.org
Mon Oct 13 13:50:10 UTC 2025
On Mon, 13 Oct 2025 09:54:31 GMT, Michael McMahon <michaelm at openjdk.org> wrote:
> Possibly. If we are agreed that 100 Continue is the only potential practical use case for this
as the PR stands now, even that is no longer relevant, because of the auto-send.
We are in one accord then that this PR implementation does not have anything of worth.
> Continue considering that the server already handles it. Is there a use case for allowing the user to send it separately?
Given that the server automatically sends 100-Continue without allowing the handler to validate if the body should be sent, I think we should make a different PR delaying the auto send until `getRequestBody().read()`. This would allow for proper 100 continue behavior where the server only continues the request if the request is validated.
> If we're not going to allow sendResponseHeaders to accept 1xx codes
I'm amenable to banning 100 (because the server handles it) and 103 (needs http2), but once again I ask for a discussion on http 101 before we proceed to say that all 1xx codes are not accepted.
I've created #27751 for review and a place to discuss, would you kindly take a moment to review and voice your concerns?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/27069#issuecomment-3397611795
More information about the net-dev
mailing list