azure-pipelines-task-lib 4.15.0 → 4.17.0

package/internal.js

@@ -174,7 +174,6 @@ function _setResourcePath(path, ignoreWarnings) {
     }
     else {
         if (ignoreWarnings) {
-            _debug(_loc('LIB_ResourceFileAlreadySet', path));
         }
         else {
             _warning(_loc('LIB_ResourceFileAlreadySet', path), IssueSource.TaskInternal);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azure-pipelines-task-lib",
-  "version": "4.15.0",
+  "version": "4.17.0",
   "description": "Azure Pipelines Task SDK",
   "main": "./task.js",
   "typings": "./task.d.ts",
@@ -31,7 +31,7 @@
     "minimatch": "3.0.5",
     "nodejs-file-downloader": "^4.11.1",
     "q": "^1.5.1",
-    "semver": "^5.1.0",
+    "semver": "^5.7.2",
     "shelljs": "^0.8.5",
     "uuid": "^3.0.1"
   },

package/toolrunner.d.ts

@@ -167,5 +167,5 @@ export declare 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.
      */
-    killChildProcess(): void;
+    killChildProcess(signal?: number | NodeJS.Signals): void;
 }

package/toolrunner.js

@@ -578,12 +578,14 @@ var ToolRunner = /** @class */ (function (_super) {
             }
             //pipe stdout of first tool to stdin of second tool
             (_a = cpFirst.stdout) === null || _a === void 0 ? void 0 : _a.on('data', function (data) {
-                var _a;
+                var _a, _b;
                 try {
                     if (fileStream) {
                         fileStream.write(data);
                     }
-                    (_a = cp.stdin) === null || _a === void 0 ? void 0 : _a.write(data);
+                    if (!((_a = cp.stdin) === null || _a === void 0 ? void 0 : _a.destroyed)) {
+                        (_b = cp.stdin) === null || _b === void 0 ? void 0 : _b.write(data);
+                    }
                 }
                 catch (err) {
                     _this._debug('Failed to pipe output of ' + toolPathFirst + ' to ' + toolPath);
@@ -1038,14 +1040,14 @@ var ToolRunner = /** @class */ (function (_super) {
         cp.on('exit', function (code, signal) {
             state.processExitCode = code;
             state.processExited = true;
-            _this._debug("Exit code " + code + " received from tool '" + _this.toolPath + "'");
+            _this._debug("STDIO streams have closed and received exit code " + code + " and signal " + signal + " for tool '" + _this.toolPath + "'");
             state.CheckComplete();
         });
         cp.on('close', function (code, signal) {
             state.processExitCode = code;
             state.processExited = true;
             state.processClosed = true;
-            _this._debug("STDIO streams have closed for tool '" + _this.toolPath + "'");
+            _this._debug("STDIO streams have closed and received exit code " + code + " and signal " + signal + " for tool '" + _this.toolPath + "'");
             state.CheckComplete();
         });
         return new Promise(emitDoneEvent);
@@ -1149,14 +1151,14 @@ var ToolRunner = /** @class */ (function (_super) {
         cp.on('exit', function (code, signal) {
             state.processExitCode = code;
             state.processExited = true;
-            _this._debug("Exit code " + code + " received from tool '" + _this.toolPath + "'");
+            _this._debug("STDIO streams have closed and received exit code " + code + " and signal " + signal + " for tool '" + _this.toolPath + "'");
             state.CheckComplete();
         });
         cp.on('close', function (code, signal) {
             state.processExitCode = code;
             state.processExited = true;
             state.processClosed = true;
-            _this._debug("STDIO streams have closed for tool '" + _this.toolPath + "'");
+            _this._debug("STDIO streams have closed and received exit code " + code + " and signal " + signal + " for tool '" + _this.toolPath + "'");
             state.CheckComplete();
         });
         return defer.promise;
@@ -1199,9 +1201,11 @@ var ToolRunner = /** @class */ (function (_super) {
      * Used to close child process by sending SIGNINT signal.
      * It allows executed script to have some additional logic on SIGINT, before exiting.
      */
-    ToolRunner.prototype.killChildProcess = function () {
+    ToolRunner.prototype.killChildProcess = function (signal) {
+        if (signal === void 0) { signal = "SIGTERM"; }
         if (this.childProcess) {
-            this.childProcess.kill();
+            this._debug("[killChildProcess] Signal " + signal + " received");
+            this.childProcess.kill(signal);
         }
     };
     return ToolRunner;