<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi<br>
please oblige and review the current version of the fix for<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8134577">https://bugs.openjdk.java.net/browse/JDK-8134577</a><br>
<br>
at<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~msheppar/8134577/webrev.05/">http://cr.openjdk.java.net/~msheppar/8134577/webrev.05/</a><br>
<br>
which is based on feedback from the second review.<br>
<br>
In summary the following changes have been made<br>
<br>
* hosts file mapping structure has been changed to align with
/etc/hosts, that is, <ip address> <host alias list>
rather than <hostname> <ip address><br>
<br>
* elementary comment handling added, skip lines starting with #<br>
<br>
* lookupAllHostAddr returns an array of InetAddresses, rather than a
single element array containing the first host address mapping<br>
<br>
* the illegal_state_exception token handling has been removed, and
a corresponding test deadlock/Hang.java removed<br>
<br>
* transient key word added to member variables
useLocalHostsFileLookup and hostsFileName<br>
<br>
* added an explicit test to exercise the internal NameService<br>
<br>
regards<br>
Mark<br>
<br>
<div class="moz-cite-prefix">On 13/11/2015 12:44, Mark Sheppard
wrote:<br>
</div>
<blockquote cite="mid:5645DB41.8000704@oracle.com" type="cite">
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
Hi,<br>
to summarize the amendments below:<br>
<br>
ServiceConfigurationError removed - the code no longer reads the
nameservice provider property, it silently ignored<br>
<br>
try-with-resources statement added in getHostByAddr and
lookupAllHostAddr methods of NameService, and duplicate statements
removed<br>
<br>
hosts file name in tests changed to TestHosts to avoid any
potential confusion with /etc/hosts<br>
<br>
regards<br>
Mark<br>
<br>
<br>
<br>
<div class="moz-cite-prefix">On 12/11/2015 16:46, Mark Sheppard
wrote:<br>
</div>
<blockquote cite="mid:5644C26B.2030007@oracle.com" type="cite">
<meta content="text/html; charset=utf-8"
http-equiv="Content-Type">
Hi,<br>
based on feedback from first review the updates have been
amended<br>
please oblige and review the current set of changes as per<br>
<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Emsheppar/8134577/webrev.02/">http://cr.openjdk.java.net/~msheppar/8134577/webrev.02/</a><br>
<br>
regards<br>
Mark<br>
<br>
<div class="moz-cite-prefix">On 25/10/2015 23:32, Mark Sheppard
wrote:<br>
</div>
<blockquote cite="mid:562D6679.9090509@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html;
charset=utf-8">
Hi,<br>
please oblige and review the following changes<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Emsheppar/8134577/webrev/">http://cr.openjdk.java.net/~msheppar/8134577/webrev/</a><br>
<br>
which address the issue raised in<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8134577">https://bugs.openjdk.java.net/browse/JDK-8134577</a><br>
<br>
the operative word has been "eliminate".<br>
As such, the interface and service descriptor
sun.net.spi.nameservice.NameService <br>
sun.net.spi.nameservice.NameServiceDescriptor<font color="red"><b><br>
</b></font>together with its implementation
(sun.net.nameservice.dns.DNSNameService)<br>
has been remove from the JDK libraries.<br>
<br>
The immediate impact is seen in the JDK testing framework.<br>
<br>
To facilitate testing activity, and provide a replacement for
the customized NameService implementations in the<br>
JDK tests, the default NameService has been extended to
support <br>
the retrieval of host to IP address mappings from a file. <br>
The file path is specified with a system property "
jdk.internal.hosts".<br>
<br>
Previously a nameservice provider was specified by setting the
system property<br>
"sun.net.spi.nameservice.provider.", as per the documentation<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://docs.oracle.com/javase/8/docs/technotes/guides/net/properties.html">http://docs.oracle.com/javase/8/docs/technotes/guides/net/properties.html</a><br>
<br>
InetAddress now tests to determine if this property is set and
will throw a ServiceConfigurationError<br>
indicating that this functionality is no longer supported. The
choice of ServideConfigurationError may cause<br>
some debate, or disagreement. The rationale was that
InternalError, is documented to relate to a JVM error,<br>
and javax.naming.NamingException has a context of DirContext.
<br>
A possible alternative candidate could be
javax.naming.ServiceUnavailableException.<br>
As such, the setting of the property
"sun.net.spi.nameservice.provider." was used, previously, as a
configuration<br>
parameter for the loading of a NamerService service provider,
and as this is now (considered) an error,
ServiceConfigurationError, <br>
seemed a best fit!<br>
<br>
These changes impacted a number of jdk security tests, also.
The affected tetsts have been amended to adopt the<br>
changes, with the exception of
test/sun/security/x509/URICertStore/ExtensionsWithLDAP.java <br>
which will require some rewrite.<br>
<br>
regards<br>
Mark<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>