[prev in list] [next in list] [prev in thread] [next in thread]
List: tomcat-dev
Subject: DO NOT REPLY [Bug 29727] -
From: bugzilla () apache ! org
Date: 2004-10-30 6:03:51
Message-ID: 20041030060351.16428.qmail () nagoya ! betaversion ! org
[Download RAW message or body]
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 ia_rud@mail.ru 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: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic