RFR: 8361060: Keep track of the origin server against which a jdk.internal.net.http.HttpConnection was constructed [v2]

Jaikiran Pai jpai at openjdk.org
Mon Jun 30 15:23:22 UTC 2025


On Mon, 30 Jun 2025 11:28:49 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request incrementally with four additional commits since the last revision:
>> 
>>  - SNI server names can now be derived from the Origin instance
>>  - strip the square brackets from URI's host when constructing an Origin
>>  - support only lower case http and https literals for scheme in Origin
>>  - add new line
>
> src/java.net.http/share/classes/jdk/internal/net/http/Origin.java line 48:
> 
>> 46:             throw new IllegalArgumentException("Invalid port");
>> 47:         }
>> 48:     }
> 
> Should we enforce lower case for scheme and host in this constructor?
> For instance - convert to lower case if needed in `from(URI)` and throw/assert here if not lower case?

We only support `http` and `https`. So I think it makes sense to convert it to lower case and expect the scheme to be either of these. I've updated the PR accordingly.

> src/java.net.http/share/classes/jdk/internal/net/http/Origin.java line 94:
> 
>> 92:         return host + ":" + port;
>> 93:     }
>> 94: }
> 
> Missing newline.

Fixed

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26041#discussion_r2175338532
PR Review Comment: https://git.openjdk.org/jdk/pull/26041#discussion_r2175335449


More information about the net-dev mailing list