only one slack message per signal catch, #220

phetsims · Nov 1, 2024 · da2d17c · da2d17c
1 parent 9c0f8a2
commit da2d17c
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/js/server/QuickServer.ts b/js/server/QuickServer.ts
@@ -140,17 +140,21 @@ class QuickServer {
    * Send a slack message when exiting unexpectedly to say that we exited.
    */
   private wireUpMessageOnExit(): void {
+    let handledSignal = false;
 
     // catching signals and do something before exit
     [ 'SIGINT', 'SIGHUP', 'SIGQUIT', 'SIGILL', 'SIGTRAP', 'SIGABRT', 'SIGBUS', 'SIGFPE', 'SIGUSR1',
       'SIGSEGV', 'SIGUSR2', 'SIGTERM', 'beforeExit', 'uncaughtException', 'unhandledRejection'
     ].forEach( sig => {
       process.on( sig, ( error: Error ) => {
-        const message = `CTQ has caught ${sig} and will now exit. ${error}`;
-        winston.info( message );
-        this.slackMessage( message ).then( () => {
-          process.exit( 1 );
-        } ).catch( e => { throw e; } );
+        if ( !handledSignal ) {
+          handledSignal = true;
+          const message = `CTQ has caught ${sig} and will now exit. ${error}`;
+          winston.info( message );
+          this.slackMessage( message ).then( () => {
+            process.exit( 1 );
+          } ).catch( e => { throw e; } );
+        }
       } );
     } );
   }