On Tue, 16 Dec 2025 17:18:18 GMT, Daniel Jeliński <[email protected]> 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
