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

Zhengyu Gu zgu at redhat.com
Fri Aug 7 12:47:53 UTC 2020


Hi Paul,

Based on Alan Bateman's comment, we can not backport public API to 8u. I 
guess LdapDnsProvider.java and LdapDnsProviderResult.java have to be 
moved to com.sun.jndi.ldap and com.sun.jndi.ldap.spi packages.

I will withdraw this CSR, ok?

Thanks,

-Zhengyu





On 8/6/20 5:32 PM, Hohensee, Paul wrote:
> I filed a backport issue
> 
> https://bugs.openjdk.java.net/browse/JDK-8251270
> 
> and corresponding CSR
> 
> https://bugs.openjdk.java.net/browse/JDK-8251270
> 
> and assigned them to Zhengyu.
> 
> On 8/6/20, 1:50 PM, "jdk8u-dev on behalf of Hohensee, Paul" <jdk8u-dev-retn at openjdk.java.net on behalf of hohensee at amazon.com> wrote:
> 
>      +1 on needing a CSR for the backport. I'd missed the CSR link because it was buried under "Show 5 more links".
> 
>      I posted an RFO (Request for Opinion) on the same topic of approving strictly additive CSRs for backport.
> 
>      https://mail.openjdk.java.net/pipermail/jdk8u-dev/2020-August/012319.html
> 
>      Thanks,
>      Paul
> 
>      On 8/6/20, 8:02 AM, "jdk8u-dev on behalf of Severin Gehwolf" <jdk8u-dev-retn at openjdk.java.net on behalf of sgehwolf at redhat.com> wrote:
> 
>          On Thu, 2020-08-06 at 10:13 -0400, Zhengyu Gu wrote:
>          > > 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.
>          >
>          > It does not seem to have a CSR for 11u backport(?) Do 8u need one?
> 
>          Right. I don't know about what happened there for Oracle JDK. You could
>          try to ask.
> 
>          I believe we need one since this is changing the 8 SE API by adding:
> 
>          javax/naming/ldap/spi/LdapDnsProvider.java
>          javax/naming/ldap/spi/LdapDnsProviderResult.java
> 
>          Thanks,
>          Severin
> 
> 
> 



More information about the jdk8u-dev mailing list