Catch & ignore ENOTCONN errors when piping Cypress subprocess (#5293)

* Catch & ignore ENOTCONN errors when piping Cypress subprocess

* Update https-proxy-agent to point back to original repo

cli/lib/exec/spawn.js

@@ -179,13 +179,14 @@ module.exports = {
         }
 
         // https://github.com/cypress-io/cypress/issues/1841
+        // https://github.com/cypress-io/cypress/issues/5241
         // In some versions of node, it will throw on windows
         // when you close the parent process after piping
         // into the child process. unpiping does not seem
         // to have any effect. so we're just catching the
         // error here and not doing anything.
         process.stdin.on('error', (err) => {
-          if (err.code === 'EPIPE') {
+          if (['EPIPE', 'ENOTCONN'].includes(err.code)) {
             return
           }
 

cli/test/lib/exec/spawn_spec.js

@@ -439,24 +439,28 @@ describe('lib/exec/spawn', function () {
       })
     })
 
-    it('catches process.stdin errors and returns when code=EPIPE', function () {
-      this.spawnedProcess.on.withArgs('close').yieldsAsync(0)
+    // https://github.com/cypress-io/cypress/issues/1841
+    // https://github.com/cypress-io/cypress/issues/5241
+    ;['EPIPE', 'ENOTCONN'].forEach((errCode) => {
+      it(`catches process.stdin errors and returns when code=${errCode}`, function () {
+        this.spawnedProcess.on.withArgs('close').yieldsAsync(0)
 
-      return spawn.start()
-      .then(() => {
-        let called = false
+        return spawn.start()
+        .then(() => {
+          let called = false
 
-        const fn = () => {
-          called = true
-          const err = new Error()
+          const fn = () => {
+            called = true
+            const err = new Error()
 
-          err.code = 'EPIPE'
+            err.code = errCode
 
-          return process.stdin.emit('error', err)
-        }
+            return process.stdin.emit('error', err)
+          }
 
-        expect(fn).not.to.throw()
-        expect(called).to.be.true
+          expect(fn).not.to.throw()
+          expect(called).to.be.true
+        })
       })
     })