On Fri, 20 Oct 2023 12:00:51 GMT, Daniel Jeliński <[email protected]> wrote:

>> This patch fixes the encoding of IPv6 addresses in the URI provided to 
>> ProxySelector.
>> 
>> The original code percent-encoded the square brackets added around the IPv6 
>> addresses, producing URIs that could not be parsed correctly.
>> 
>> The proposed code does not percent-encode the IPv6 addresses at all.
>> 
>> Added a test to verify that different forms of addresses are processed 
>> correctly. The test verifies the handling of IPv4 addresses, IPv6 without 
>> scope, IPv6 with numeric scope, IPv6 with textual scope (only if present on 
>> the machine running tests), hostnames and FQDNs
>> 
>> Tier1-3 tests passed.
>
> Daniel Jeliński has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Remove redundant checks

Updated the test to run in othervm and removed the code to restore original 
ProxySelector.

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

PR Comment: https://git.openjdk.org/jdk/pull/16265#issuecomment-1775319873

Reply via email to