net: check objectMode first and then readble || writable

Co-authored-by: Luigi Pinca <[email protected]> PR-URL: #40344 Fixes: #40336 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Minwoo Jung <[email protected]> Reviewed-By: Zijian Liu <[email protected]> Reviewed-By: Robert Nagy <[email protected]>
nodejs · Oct 13, 2021 · c7cd8ef · c7cd8ef
1 parent 4644662
commit c7cd8ef
Showing 1 changed file with 15 additions and 14 deletions.
diff --git a/lib/net.js b/lib/net.js
@@ -283,20 +283,21 @@ const kSetNoDelay = Symbol('kSetNoDelay');
 
 function Socket(options) {
   if (!(this instanceof Socket)) return new Socket(options);
-  const invalidKeys = [
-    'objectMode',
-    'readableObjectMode',
-    'writableObjectMode',
-  ];
-  invalidKeys.forEach((invalidKey) => {
-    if (ObjectKeys(options).includes(invalidKey)) {
-      throw new ERR_INVALID_ARG_VALUE(
-        `options.${invalidKey}`,
-        options[invalidKey],
-        'is not supported'
-      );
-    }
-  });
+  if (options.objectMode) {
+    throw new ERR_INVALID_ARG_VALUE(
+      'options.objectMode',
+      options.objectMode,
+      'is not supported'
+    );
+  } else if (options.readableObjectMode || options.writableObjectMode) {
+    throw new ERR_INVALID_ARG_VALUE(
+      `options.${
+        options.readableObjectMode ? 'readableObjectMode' : 'writableObjectMode'
+      }`,
+      options.readableObjectMode || options.writableObjectMode,
+      'is not supported'
+    );
+  }
 
   this.connecting = false;
   // Problem with this is that users can supply their own handle, that may not