fix(security): increase buffer size for npm ls outputs

Author: Andras Toth

lib/agent/security/security.js

@@ -8,6 +8,8 @@ var uniq = require('lodash.uniq')
 var inherits = require('util').inherits
 var Agent = require('../agent')
 
+var MAX_BUFFER = 1024 * 1024 // 1 mb
+
 function Security (options) {
   this.collectorApi = options.collectorApi
   Agent.call(this, 'Security')
@@ -48,7 +50,9 @@ Security.prototype.collectSnykFlags = function collectSnykFlags (callback) {
         return patch.split(':')[1]
       })
 
-    childProcess.execFile('npm', ['ls'].concat(uniq(packageNames)).concat(['--parseable', '--long']), function (_error, stdout, stderr) {
+    childProcess.execFile('npm', ['ls'].concat(uniq(packageNames)).concat(['--parseable', '--long']), {
+      maxBuffer: MAX_BUFFER
+    }, function (_error, stdout, stderr) {
       if (_error) {
         // ignore
         debug.error('collectSnykFlags',
@@ -110,9 +114,8 @@ Security.prototype.collectSnykFlags = function collectSnykFlags (callback) {
 }
 
 Security.prototype.collectDependencies = function collectDependencies (callback) {
-  var maxBuffer = 10 * 1024 * 1024 // 10mb
   childProcess.execFile('npm', ['ls', '--json', '--production'], {
-    maxBuffer: maxBuffer
+    maxBuffer: MAX_BUFFER
   }, function (error, stdout, stderr) {
     if (error) {
       // ignore