deps: upgrade npm to 5.8.0

PR-URL: https://github.com/nodejs/node/pull/19560
Fixes: https://github.com/nodejs/node/issues/19271
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>

1 files changed, 3 insertions, 1 deletions

diff --git a/deps/npm/node_modules/mississippi/node_modules/pump/index.js b/deps/npm/node_modules/mississippi/node_modules/pump/index.js
index 060ce5f4fd..c15059f176 100644
--- a/deps/npm/node_modules/mississippi/node_modules/pump/index.js
+++ b/deps/npm/node_modules/mississippi/node_modules/pump/index.js
@@ -3,12 +3,14 @@ var eos = require('end-of-stream')
 var fs = require('fs') // we only need fs to get the ReadStream and WriteStream prototypes
 
 var noop = function () {}
+var ancient = /^v?\.0/.test(process.version)
 
 var isFn = function (fn) {
   return typeof fn === 'function'
 }
 
 var isFS = function (stream) {
+  if (!ancient) return false // newer node version do not need to care about fs is a special way
   if (!fs) return false // browser
   return (stream instanceof (fs.ReadStream || noop) || stream instanceof (fs.WriteStream || noop)) && isFn(stream.close)
 }
@@ -37,7 +39,7 @@ var destroyer = function (stream, reading, writing, callback) {
     if (destroyed) return
     destroyed = true
 
-    if (isFS(stream)) return stream.close() // use close for fs streams to avoid fd leaks
+    if (isFS(stream)) return stream.close(noop) // use close for fs streams to avoid fd leaks
     if (isRequest(stream)) return stream.abort() // request.destroy just do .end - .abort is what we want
 
     if (isFn(stream.destroy)) return stream.destroy()