Add terminations to exec

microsoft · Jan 8, 2024 · 09724af · 09724af
1 parent 644e9df
commit 09724af
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/node/task.ts b/node/task.ts
@@ -1508,8 +1508,8 @@ export function exec(tool: string, args: any, options?: trm.IExecOptions): Q.Pro
         }
     }
 
-    if (getBoolFeatureFlag("ENABLE_KILL_CHILD_PROCESS")) {
-        ['SIGINT', 'SIGTERM'].forEach(x => {
+    if (getBoolFeatureFlag('ENABLE_KILL_CHILD_PROCESS')) {
+        ['SIGINT', 'SIGTERM'].forEach((x: NodeJS.Signals) => {
             tr.on(x, () => {
                 tr.killChildProcess(x);
             });

diff --git a/node/toolrunner.ts b/node/toolrunner.ts
@@ -1343,9 +1343,9 @@ export class ToolRunner extends events.EventEmitter {
      * Used to close child process by sending SIGNINT signal.
      * It allows executed script to have some additional logic on SIGINT, before exiting.
      */
-    public killChildProcess(): void {
+    public killChildProcess(signal?: number | NodeJS.Signals | undefined): void {
         if (this.childProcess) {
-            this.childProcess.kill();
+            this.childProcess.kill(signal);
         }
     }
 }