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");
     }