RFR: 8349670: HttpServer: sending interim responses fails after JDK-7026262 [v8]
robert engels
duke at openjdk.org
Thu Oct 9 04:37:04 UTC 2025
On Fri, 3 Oct 2025 10:23:55 GMT, Michael McMahon <michaelm at openjdk.org> wrote:
>>> I intend to do a more thorough review in the coming days.
>>
>> I'm sure that when you get time to review in earnest, you'll find it quite hard to believe that these changes have any relation.
>
> I'm okay with @SentryMan fixing this. I filed another issue for the API change that I suggested (https://bugs.openjdk.org/browse/JDK-8368955) and I will open a PR for that after this issue is fixed.
> For me that's pretty much the long and short of it, but I took another look at the jira issue and @Michael-Mc-Mahon did seem to mention an issue with the current implementation.
>
> >
I don't think this is an issue. The continue is sent before the request body is read. What you can't do is other inspection of the headers to maybe error the request instead of sending the continue.
The recent PR to only send the continue when the input stream is read fixes that.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/27069#issuecomment-3384085154
More information about the net-dev
mailing list