azure-pipelines-task-lib - npm Package Compare versions

Comparing version 4.17.1 to 4.17.2

package.json

 {
   "name": "azure-pipelines-task-lib",
-  "version": "4.17.1",
+  "version": "4.17.2",
   "description": "Azure Pipelines Task SDK",
@@ -5,0 +5,0 @@ "main": "./task.js",

toolrunner.js

@@ -139,14 +139,13 @@ "use strict";
     };
-    ToolRunner.prototype._processLineBuffer = function (data, strBuffer, onLine) {
+    ToolRunner.prototype._processLineBuffer = function (data, buffer, onLine) {
+        var newBuffer = buffer + data.toString();
         try {
-            var s = strBuffer + data.toString();
-            var n = s.indexOf(os.EOL);
-            while (n > -1) {
-                var line = s.substring(0, n);
+            var eolIndex = newBuffer.indexOf(os.EOL);
+            while (eolIndex > -1) {
+                var line = newBuffer.substring(0, eolIndex);
                 onLine(line);
                 // the rest of the string ...
-                s = s.substring(n + os.EOL.length);
-                n = s.indexOf(os.EOL);
+                newBuffer = newBuffer.substring(eolIndex + os.EOL.length);
+                eolIndex = newBuffer.indexOf(os.EOL);
             }
-            strBuffer = s;
         }
@@ -157,2 +156,3 @@ catch (err) {
         }
+        return newBuffer;
     };
@@ -643,3 +643,3 @@ /**
             });
-            var stdbuffer = '';
+            var stdLineBuffer = '';
             (_c = cp.stdout) === null || _c === void 0 ? void 0 : _c.on('data', function (data) {
@@ -650,7 +650,7 @@ _this.emit('stdout', data);
                 }
-                _this._processLineBuffer(data, stdbuffer, function (line) {
+                stdLineBuffer = _this._processLineBuffer(data, stdLineBuffer, function (line) {
                     _this.emit('stdline', line);
                 });
             });
-            var errbuffer = '';
+            var errLineBuffer = '';
             (_d = cp.stderr) === null || _d === void 0 ? void 0 : _d.on('data', function (data) {
@@ -663,3 +663,3 @@ _this.emit('stderr', data);
                 }
-                _this._processLineBuffer(data, errbuffer, function (line) {
+                errLineBuffer = _this._processLineBuffer(data, errLineBuffer, function (line) {
                     _this.emit('errline', line);
@@ -679,7 +679,7 @@ });
                 returnCode = code;
-                if (stdbuffer.length > 0) {
-                    _this.emit('stdline', stdbuffer);
+                if (stdLineBuffer.length > 0) {
+                    _this.emit('stdline', stdLineBuffer);
                 }
-                if (errbuffer.length > 0) {
-                    _this.emit('errline', errbuffer);
+                if (errLineBuffer.length > 0) {
+                    _this.emit('errline', errLineBuffer);
                 }
@@ -825,3 +825,3 @@ if (code != 0 && !optionsNonNull.ignoreReturnCode) {
         });
-        var stdbuffer = '';
+        var stdLineBuffer = '';
         (_c = cp.stdout) === null || _c === void 0 ? void 0 : _c.on('data', function (data) {
@@ -832,7 +832,7 @@ _this.emit('stdout', data);
             }
-            _this._processLineBuffer(data, stdbuffer, function (line) {
+            stdLineBuffer = _this._processLineBuffer(data, stdLineBuffer, function (line) {
                 _this.emit('stdline', line);
             });
         });
-        var errbuffer = '';
+        var errLineBuffer = '';
         (_d = cp.stderr) === null || _d === void 0 ? void 0 : _d.on('data', function (data) {
@@ -845,3 +845,3 @@ _this.emit('stderr', data);
             }
-            _this._processLineBuffer(data, errbuffer, function (line) {
+            errLineBuffer = _this._processLineBuffer(data, errLineBuffer, function (line) {
                 _this.emit('errline', line);
@@ -861,7 +861,7 @@ });
             returnCode = code;
-            if (stdbuffer.length > 0) {
-                _this.emit('stdline', stdbuffer);
+            if (stdLineBuffer.length > 0) {
+                _this.emit('stdline', stdLineBuffer);
             }
-            if (errbuffer.length > 0) {
-                _this.emit('errline', errbuffer);
+            if (errLineBuffer.length > 0) {
+                _this.emit('errline', errLineBuffer);
             }
@@ -981,12 +981,11 @@ if (code != 0 && !optionsNonNull.ignoreReturnCode) {
         });
-        var stdbuffer = '';
-        var errbuffer = '';
+        var stdLineBuffer = '';
+        var errLineBuffer = '';
         var emitDoneEvent = function (resolve, reject) {
-            var _this = this;
             state.on('done', function (error, exitCode) {
-                if (stdbuffer.length > 0) {
-                    _this.emit('stdline', stdbuffer);
+                if (stdLineBuffer.length > 0) {
+                    _this.emit('stdline', stdLineBuffer);
                 }
-                if (errbuffer.length > 0) {
-                    _this.emit('errline', errbuffer);
+                if (errLineBuffer.length > 0) {
+                    _this.emit('errline', errLineBuffer);
                 }
@@ -1032,3 +1031,3 @@ if (cp) {
             }
-            _this._processLineBuffer(data, stdbuffer, function (line) {
+            stdLineBuffer = _this._processLineBuffer(data, stdLineBuffer, function (line) {
                 _this.emit('stdline', line);
@@ -1044,3 +1043,3 @@ });
             }
-            _this._processLineBuffer(data, errbuffer, function (line) {
+            errLineBuffer = _this._processLineBuffer(data, errLineBuffer, function (line) {
                 _this.emit('errline', line);
@@ -1101,10 +1100,10 @@ });
         });
-        var stdbuffer = '';
-        var errbuffer = '';
+        var stdLineBuffer = '';
+        var errLineBuffer = '';
         state.on('done', function (error, exitCode) {
-            if (stdbuffer.length > 0) {
-                _this.emit('stdline', stdbuffer);
+            if (stdLineBuffer.length > 0) {
+                _this.emit('stdline', stdLineBuffer);
             }
-            if (errbuffer.length > 0) {
-                _this.emit('errline', errbuffer);
+            if (errLineBuffer.length > 0) {
+                _this.emit('errline', errLineBuffer);
             }
@@ -1147,3 +1146,3 @@ if (cp) {
             }
-            _this._processLineBuffer(data, stdbuffer, function (line) {
+            stdLineBuffer = _this._processLineBuffer(data, stdLineBuffer, function (line) {
                 _this.emit('stdline', line);
@@ -1159,3 +1158,3 @@ });
             }
-            _this._processLineBuffer(data, errbuffer, function (line) {
+            errLineBuffer = _this._processLineBuffer(data, errLineBuffer, function (line) {
                 _this.emit('errline', line);
@@ -1162,0 +1161,0 @@ });

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

429136

increased by0.07%

Worsened metrics

Lines of code

7585

decreased by-0.01%

No dependency changes