Fix SIGTERM behavior that causes systemd control of etserver to timeo…

…ut. (MisterTea#554) From various user reports, `systemctl stop et` hangs and times out requiring a SIGKILL signal. In addition to adding the hook to kill sentry/telemetry on SIGTERM, we need to exit as well. Fixes MisterTea#546 Co-authored-by: James Short <[email protected]>
jshort · Dec 19, 2022 · c31f3b9 · c31f3b9
1 parent 10dd371
commit c31f3b9
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 6 deletions.
diff --git a/codecov.yml b/codecov.yml
@@ -8,6 +8,7 @@ ignore:
   - "src/terminal/PsuedoTerminalConsole.hpp"
   - "src/terminal/PsuedoUserTerminal.hpp"
   - "src/terminal/*Main.cpp"
+  - "src/base/Headers.hpp"
   - "src/base/TcpSocketHandler*"
   - "src/base/SubprocessToString*"
 coverage:

diff --git a/src/base/Headers.hpp b/src/base/Headers.hpp
@@ -409,9 +409,16 @@ inline void HandleTerminate() {
 }
 
 inline void InterruptSignalHandler(int signum) {
-  STERROR << "Got interrupt";
   CLOG(INFO, "stdout") << endl
-                       << "Got interrupt (perhaps ctrl+c?).  Exiting." << endl;
+                       << "Got interrupt (perhaps ctrl+c?): " << signum
+                       << ".  Exiting." << endl;
+  ::exit(signum);
+}
+
+inline void TerminateSignalHandler(int signum) {
+  CLOG(INFO, "stdout") << endl
+                       << "Got terminate signal: " << signum << ".  Exiting."
+                       << endl;
   ::exit(signum);
 }
 

diff --git a/src/terminal/TelemetryService.cpp b/src/terminal/TelemetryService.cpp
@@ -135,6 +135,7 @@ TelemetryService::TelemetryService(const bool _allow,
     }
 
 #ifdef USE_SENTRY
+    cerr << "Setting up and starting sentry" << endl;
     sentry_options_t* options = sentry_options_new();
     logHttpClient->set_compress(true);
 
@@ -170,9 +171,6 @@ TelemetryService::TelemetryService(const bool _allow,
 #ifdef SIGSEGV
         SIGSEGV,
 #endif
-#ifdef SIGTERM
-        SIGTERM,
-#endif
 #ifdef SIGKILL
         SIGKILL,
 #endif
@@ -181,10 +179,18 @@ TelemetryService::TelemetryService(const bool _allow,
       signal(it, sentryShutdownHandler);
     }
 
+#ifdef SIGTERM
+    signal(SIGTERM, [](int i) {
+      // In addition to exiting like the default SIGTERM handler would do,
+      // let's shutdown sentry telemetry
+      shutdownTelemetry();
+      et::TerminateSignalHandler(i);
+    });
+#endif
 #ifdef SIGINT
     signal(SIGINT, [](int i) {
       shutdownTelemetry();
-      // Normally this is installed in TerminalServerMain, TerminalClientMain,
+      // Normally this is configured in TerminalServerMain, TerminalClientMain,
       // and TerminalMain, but we need to forward the call since Sentry
       // overrides it.  This is important to handle SIGINT from Ctrl-C.
       et::InterruptSignalHandler(i);