net: remove redundant connecting assignment

PR-URL: #43710
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>

lib/net.js

@@ -1239,7 +1239,6 @@ function afterConnect(status, handle, req, readable, writable) {
       stopPerf(self, kPerfHooksNetConnectContext);
     }
   } else {
-    self.connecting = false;
     let details;
     if (req.localAddress && req.localPort) {
       details = req.localAddress + ':' + req.localPort;

test/sequential/test-net-connect-handle-econnrefused.js

@@ -27,5 +27,6 @@ const assert = require('assert');
 const c = net.createConnection(common.PORT);
 c.on('connect', common.mustNotCall());
 c.on('error', common.mustCall((e) => {
+  assert.strictEqual(c.connecting, false);
   assert.strictEqual(e.code, 'ECONNREFUSED');
 }));