[wildfly-dev] Naming, LDAP and a little bit of JMX

[Eduardo Martins]

On Oct 16, 2013, at 5:00 PM, Darran Lofthouse <darran.lofthouse at jboss.com> wrote:

> On 16/10/13 16:50, Eduardo Martins wrote:
>> There is also WFLY-2115, but I am holding resolution till WFLY-794 is resolved.
>> 
>> For any url scheme we do not provide custom implementation, which is the case for rmi://, the code you mention is missing is actually invoked at
>> 
>> https://github.com/wildfly/wildfly/blob/master/naming/src/main/java/org/jboss/as/naming/InitialContext.java#L110
> 
> Ah I missed that line so we do attempt to resolve it that way.
> 
> Probably the bit we are missing then is the common method in the JDK code that all methods use to obtain the correct context rather than issolating it within one lookup method.
> 

Yes, that's why I believe it's not correct below ^^

>> But that's only true for lookups, which I believe is not correct, for instance a bind or InitialDirContext.search() will never go through that.
> 
> For an InitialDirContext non of the JBoss classes come into it at all, once the JDK supplied classes are found they are used - however the advanced option for finding a url context is skipped as the JDK classes assume we will handle that.

It's the same for InitialContext, you use the JDK constructors, and that's the reason we set the factory builder, otherwise we could not override the jdk naming impl.

--E