DO NOT REPLY [Bug 29727] - JNDI env-entry not reload when context reloaded

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=29727>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=29727

JNDI env-entry not reload when context reloaded





------- Additional Comments From [EMAIL PROTECTED]  2004-10-30 06:03 -------
Could you please consider the following modification of
org.apache.catalina.core.NamingContextListener class:

it was:

public void lifecycleEvent(LifecycleEvent event) {
...
            // Binding the naming context to the class loader
            if (container instanceof Context) {
                // Setting the context in read only mode
                ContextAccessController.setReadOnly(getName());
                try {
                    ContextBindings.bindClassLoader
                        (container, container,
                         ((Container)
container).getLoader().getClassLoader());
                } catch (NamingException e) {
                    log(sm.getString("naming.bindFailed", e));
                }
            }
...
}

I think, it should be:


public void lifecycleEvent(LifecycleEvent event) {
...
            // Binding the naming context to the class loader
            if (container instanceof Context) {



               namingResources.addPropertyChangeListener(this);



                // Setting the context in read only mode
                ContextAccessController.setReadOnly(getName());
                try {
                    ContextBindings.bindClassLoader
                        (container, container,
                         ((Container)
container).getLoader().getClassLoader());
                } catch (NamingException e) {
                    log(sm.getString("naming.bindFailed", e));
                }
            }
...
}


Thank you,
Igor

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]