Merge pull request payara#6749 from flowlogix/fix-dangling-session-th…

…read Properly cleaning up ContainerBackgroundSessionProcessor threads after application is undeployed
Pandrex247 · Jul 5, 2024 · 5fb6c40 · 5fb6c40
1 parent 9d95c69
commit 5fb6c40
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/appserver/web/web-core/src/main/java/org/apache/catalina/core/ContainerBase.java b/appserver/web/web-core/src/main/java/org/apache/catalina/core/ContainerBase.java
@@ -1750,7 +1750,7 @@ protected void threadStart() {
      * to update lastaccesstime and accessedTime.
      */
     protected void threadSessionStart() {
-        if (sessionThread != null)
+        if (sessionThread != null || manager == null)
             return;
         threadSessionDone = false;
         String threadName = "ContainerBackgroundSessionProcessor[" + toString() + "]";

diff --git a/appserver/web/web-core/src/main/java/org/apache/catalina/core/StandardContext.java b/appserver/web/web-core/src/main/java/org/apache/catalina/core/StandardContext.java
@@ -5946,6 +5946,8 @@ public synchronized void stop(boolean isShutdown)
 
             // Stop ContainerBackgroundProcessor thread
             super.threadStop();
+            // Stop ContainerBackgroundSessionProcessor thread
+            super.threadSessionStop();
 
             if ((manager != null) && (manager instanceof Lifecycle)) {
                 if(manager instanceof StandardManager) {