fix: auth handler hanging until timeout if errored

Author: aryanjassal

src/nodes/NodeConnectionManager.ts

@@ -1770,18 +1770,18 @@ class NodeConnectionManager {
             'Expected success message but got authentication message',
           );
         }
+        await writer.close();
+        await reader.cancel();
       });
-      // Gracefully close the connection if there was no error
-      await writer?.close();
-      await reader?.cancel();
       connectionsEntry.authenticatedForward = AuthenticatingState.SUCCESS;
     } catch (e) {
       const err = new nodesErrors.ErrorNodeManagerAuthenticationFailed(
         undefined,
         { cause: e },
       );
-      await writer?.abort(err);
-      await reader?.cancel(err);
+      // We only care if the reader and writer closed properly
+      await writer?.abort(err).catch(() => {});
+      await reader?.cancel(err).catch(() => {});
       // Make sure any pending authentication is set to FAIL accordingly
       if (
         connectionsEntry.authenticatedForward === AuthenticatingState.PENDING