tls: remove redundant code in onConnectSecure()

Remove redundant code by moving it to outside of `if/else`. PR-URL: #49457 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Matthew Aitken <[email protected]>
nodejs · Sep 4, 2023 · c8628ed · c8628ed
1 parent 8dfe424
commit c8628ed
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/lib/_tls_wrap.js b/lib/_tls_wrap.js
@@ -1688,14 +1688,12 @@ function onConnectSecure() {
     debug('client emit secureConnect. rejectUnauthorized: %s, ' +
           'authorizationError: %s', options.rejectUnauthorized,
           this.authorizationError);
-    this.secureConnecting = false;
-    this.emit('secureConnect');
   } else {
     this.authorized = true;
     debug('client emit secureConnect. authorized:', this.authorized);
-    this.secureConnecting = false;
-    this.emit('secureConnect');
   }
+  this.secureConnecting = false;
+  this.emit('secureConnect');
 
   this[kIsVerified] = true;
   const session = this[kPendingSession];