RFR: 8373677: Clear text HttpServer connection could fail fast if receiving SSL ClientHello [v2]

[Daniel Fuchs]

On Tue, 16 Dec 2025 17:18:18 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

>> Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   minor test fix - unused import + obsolete comment
>
> src/jdk.httpserver/share/classes/sun/net/httpserver/ServerImpl.java line 740:
> 
>> 738:                 Request req;
>> 739:                 try {
>> 740:                     req = new Request(rawin, rawout, newconnection && !https);
> 
> I'd skip the `!https` check here.

We only need the check for clear connections. Why would you keep it with https?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28827#discussion_r2624915078