[8u] RFR 8160768: Add capability to custom resolve host/domain names within the default JNDI LDAP provider

[Severin Gehwolf]

Hi Zhengyu,

On Wed, 2020-08-05 at 09:48 -0400, Zhengyu Gu wrote:
> Hi Paul,
> 
> Sorry for replying late. Somehow, this email slipped through the cracks, 
> and thanks Michael for bringing it to my attention.
> 
> 
> On 7/13/20 5:27 PM, Hohensee, Paul wrote:
> > The webrev looks it's corrupted. E.g., it includes the changes from 8217606, and not the InitialDirContext.java and CheckConfigs.policy changes you mention.
> 
> It had dependence on 8217606, now it is pushed, so that the webrev is 
> much cleaner.
> 
> Updated: http://cr.openjdk.java.net/~zgu/JDK-8160768-8u/jdk/webrev.01/

The original patch required a CSR[1], so we should file one for JDK 8u
too. I wonder where the CSR is for Oracle JDK 8, though.

Thanks,
Severin

[1] https://bugs.openjdk.java.net/browse/JDK-8192975

> Test:
>    Reran jdk_other.
> 
> Thanks,
> 
> -Zhengyu
> 
> 
> 
> > Paul
> > 
> > On 7/9/20, 1:39 PM, "jdk8u-dev on behalf of Zhengyu Gu" <jdk8u-dev-retn at openjdk.java.net on behalf of zgu at redhat.com> wrote:
> > 
> >      Hi,
> > 
> >      I would like to backport this patch to 8u for parity with Oracle 8u261.
> > 
> >      The original patch does not apply cleanly.
> > 
> >      Other than a couple of minor conflicts:
> > 
> >      1) Comments in InitialDirContext.java did not apply cleanly
> >      2) Unpatched CheckConfigs.policy files did not match
> > 
> >      I made following modification for 8u:
> > 
> >      1) Removed module-info.java section, it does not apply to 8u.
> >      2) LdapDnsProvider.java and LdapDnsProviderResult.java use APIs
> >      (List.copyOf() and List.of()) that do not exist in jdk8. Rewrote the
> >      code with ArrayList<String>.
> >      3) Removed @modules annotation in LdapDnsProviderTest.java
> > 
> >      Additional, I need to modify langtools to get javac to take
> >      com.sun.jndi.ldap.spi package and complain about it.
> > 
> >      Original bug: https://bugs.openjdk.java.net/browse/JDK-8160768
> >      Original patch: http://hg.openjdk.java.net/jdk/jdk/rev/a609d549992a
> > 
> >      8u jdk webrev: http://cr.openjdk.java.net/~zgu/JDK-8160768-8u/jdk/webrev.00/
> >      8u langtools webrev:
> >      http://cr.openjdk.java.net/~zgu/JDK-8160768-8u/langtools/webev.00/
> > 
> > 
> >      Test:
> >         jdk_other on Linux x86_64
> > 
> >      Thanks,
> > 
> >      -Zhengyu
> > 
> > 
> >