You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2005/01/28 00:36:13 UTC
svn commit: r128443 - /geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java
Author: djencks
Date: Thu Jan 27 15:36:11 2005
New Revision: 128443
URL: http://svn.apache.org/viewcvs?view=rev&rev=128443
Log:
better jetty web app shutdown, avoid a nasty stack trace
Modified:
geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java
Modified: geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java
Url: http://svn.apache.org/viewcvs/geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java?view=diff&rev=128443&p1=geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java&r1=128442&p2=geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java&r2=128443
==============================================================================
--- geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java (original)
+++ geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java Thu Jan 27 15:36:11 2005
@@ -266,7 +266,6 @@
super.stop();
return;
}
- jettyContainer.removeContext(this);
if (securityInterceptor != null) {
securityInterceptor.stop();
@@ -276,6 +275,7 @@
super.doStop();
} finally {
leaveContextScope(null, null, context);
+ jettyContainer.removeContext(this);
}
log.info("JettyWebAppContext stopped");
}