On Mon, 3 Mar 2025 11:18:10 GMT, SendaoYan <[email protected]> wrote:
>> Hi all,
>>
>> Two java/net/InetAddress tests fails "java.net.UnknownHostException" on some
>> special machines. The machine cannot connect to the Internet directly and
>> needs to be connected to the Internet through a jump machine. The
>> java/net/InetAddress tests report "java.net.UnknownHostException:
>> bugs.openjdk.org: Temporary failure in name resolution" failure do not
>> caused by JVM bug but environmentalk issue, so this PR match the
>> java.net.UnknownHostException and throw jtreg.SkippedException instead of
>> report test fails.
>>
>> And run the tests with JVM options `-Dhttp.proxyHost=192.168.50.1
>> -Dhttp.proxyPort=10991 -Dhttps.proxyHost=192.168.50.1
>> -Dhttps.proxyPort=10991 -DsocksProxyHost=192.168.50.1
>> -DsocksProxyPort=10991` can not work around. So I create this PR to fix the
>> test bug.
>>
>> Command wget shows that the machine should connect to internet through a
>> jumper machine(192.168.50.1:10991):
>>
>>> wget www.bing.com
>> --2025-02-24 17:56:25-- http://www.bing.com/
>> Connecting to 192.168.50.1:10991... connected.
>> Proxy request sent, awaiting response... 301 Moved Permanently
>> Location: http://cn.bing.com/ [following]
>> --2025-02-24 17:56:25-- http://cn.bing.com/
>> Reusing existing connection to 192.168.50.1:10991.
>> Proxy request sent, awaiting response... 200 OK
>> Length: 13006 (13K) [text/html]
>> Saving to: ‘index.html’
>>
>>
>> Change has been verified locally, test-fix only and make tests more
>> robustness, no risk.
>
> SendaoYan has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Remove "@build jtreg.SkippedException"
test/jdk/java/net/InetAddress/getOriginalHostName.java line 47:
> 45: final String HOST = "dummyserver.java.net";
> 46: InetAddress ia = null;
> 47: ia = InetAddress.getByName(HOST);
The try-catch should be around this call only. Stg like:
try {
ia = InetAddress.getByName(HOST);
} catch (UnknownHostException uhe) {
System.out.println("Skipping " + HOST + " due to " + uhe;
}
if (ia != null) testInetAddress(ia, HOST);
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23767#discussion_r1977624518
