On 25/10/2015 23:32, Mark Sheppard wrote:
Hi,
please oblige and review the following changes
http://cr.openjdk.java.net/~msheppar/8134577/webrev/
which address the issue raised in
https://bugs.openjdk.java.net/browse/JDK-8134577
the operative word has been "eliminate".
This has been a very troublesome mechanism so good to see this reduced
down to a hosts file that we can we use for testing.
Shouldn't the reading of sun.net.spi.nameservice.provider.<N> be removed
so that the throwing of ServiceConfigurationError can be removed too?
Also just on the property name, I would assume it should be something
like "jdk.net.hosts.file" rather than "jdk.internal.hosts".
Do we have tests that are using the illegal_state_exception token? I
didn't spot any in the webrev. Just wondering if this is the right
exception for testing.
-Alan.
