Issue #8088 Add STOP.EXIT System property to configure ShutdownMonito…

…r.exitVM
jetty · Jun 1, 2022 · 74c7985 · 74c7985
1 parent 991ff94
commit 74c7985
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 2 deletions.
diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/ShutdownMonitor.java b/jetty-server/src/main/java/org/eclipse/jetty/server/ShutdownMonitor.java
@@ -90,7 +90,7 @@ public static boolean isRegistered(LifeCycle lifeCycle)
     private final String host;
     private int port;
     private String key;
-    private boolean exitVm;
+    private boolean exitVm = true;
     private boolean alive;
 
     /**
@@ -107,7 +107,7 @@ private ShutdownMonitor()
         this.host = System.getProperty("STOP.HOST", "127.0.0.1");
         this.port = Integer.getInteger("STOP.PORT", -1);
         this.key = System.getProperty("STOP.KEY", null);
-        this.exitVm = true;
+        this.exitVm = Boolean.getBoolean("STOP.EXIT");
     }
 
     private void addLifeCycles(LifeCycle... lifeCycles)
@@ -308,6 +308,8 @@ private ServerSocket listen()
             // establish the port and key that are in use
             debug("STOP.PORT=%d", port);
             debug("STOP.KEY=%s", key);
+            //also show if we're exiting the jvm or not
+            debug("STOP.EXIT=%b", exitVm);
         }
     }
 

diff --git a/jetty-server/src/test/java/org/eclipse/jetty/server/ShutdownMonitorTest.java b/jetty-server/src/test/java/org/eclipse/jetty/server/ShutdownMonitorTest.java
@@ -32,6 +32,7 @@
 import org.junit.jupiter.api.Test;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
@@ -120,6 +121,38 @@ private void testStartStop(boolean reusePort) throws Exception
         assertTrue(!monitor.isAlive());
     }
 
+    @Test
+    public void testNoExitSystemProperty() throws Exception
+    {
+        System.setProperty("STOP.EXIT", "false");
+        ShutdownMonitor monitor = ShutdownMonitor.getInstance();
+        monitor.setPort(0);
+        assertFalse(monitor.isExitVm());
+        monitor.start();
+
+        try (CloseableServer server = new CloseableServer())
+        {
+            server.setStopAtShutdown(true);
+            server.start();
+
+            //shouldn't be registered for shutdown on jvm
+            assertTrue(ShutdownThread.isRegistered(server));
+            assertTrue(ShutdownMonitor.isRegistered(server));
+
+            String key = monitor.getKey();
+            int port = monitor.getPort();
+
+            stop("stop", port, key, true);
+            monitor.await();
+
+            assertTrue(!monitor.isAlive());
+            assertTrue(server.stopped);
+            assertTrue(!server.destroyed);
+            assertTrue(!ShutdownThread.isRegistered(server));
+            assertTrue(!ShutdownMonitor.isRegistered(server));
+        }
+    }
+
     @Test
     public void testForceStopCommand() throws Exception
     {