On Tue, 28 Apr 2026 16:44:00 GMT, Daniel Fuchs <[email protected]> wrote:
>> Volkan Yazici has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Big facelift
>
> src/java.net.http/share/classes/jdk/internal/net/http/AltSvcProcessor.java
> line 66:
>
>> 64: private static final boolean allowLocalHostOrigin =
>> ALTSVC_ALLOW_LOCAL_HOST_ORIGIN;
>> 65:
>> 66: @SuppressWarnings("deprecation")
>
> Would the new factory method throw here if we call it with "localhost"? If
> it does not it would be preferable to call it than use `@SW`.
I was told by @AlanBateman in [this conversation] to carry out the migration to
the new methods in a separate PR.
[this conversation]:
https://github.com/openjdk/jdk/pull/30747#discussion_r3122066569
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30747#discussion_r3166676157
