Skipping new nodes which do not have the http property present yet.

Signed-off-by: Jaco Smit <jacobuss@amazon.com>

CHANGELOG.md

@@ -14,6 +14,8 @@ Inspired by [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 ## [3.3.0]
 ### Changed
 - Updated API to match the OpenSearch spec from Feb 11, 2025
+### Fixed
+- Skip nodes that are not ready ready yet ([#984](https://github.com/opensearch-project/opensearch-js/issues/984))
 
 ## [3.2.0]
 ### Changed

lib/pool/BaseConnectionPool.js

@@ -222,6 +222,12 @@ class BaseConnectionPool {
 
     for (let i = 0, len = ids.length; i < len; i++) {
       const node = nodes[ids[i]];
+
+      // New nodes do not have the http property populated yet. Skip this for now.
+      if (node.http === undefined) {
+        continue;
+      }
+
       // If there is no protocol in
       // the `publish_address` new URL will throw
       // the publish_address can have two forms:

test/unit/base-connection-pool.test.js

@@ -628,5 +628,36 @@ test('API', (t) => {
     }
   });
 
+  t.test('Should skip nodes when the http property is undefined', (t) => {
+    const pool = new BaseConnectionPool({ Connection });
+    const nodes = {
+      a1: {
+        http: {
+          publish_address: '127.0.0.1:9200',
+        },
+        roles: ['master', 'data', 'ingest'],
+      },
+      a2: {
+        roles: ['master', 'data', 'ingest'],
+      },
+    };
+
+    t.same(pool.nodesToHost(nodes, 'http:'), [
+      {
+        url: new URL('http://127.0.0.1:9200'),
+        id: 'a1',
+        roles: {
+          master: true,
+          data: true,
+          ingest: true,
+        },
+      },
+    ]);
+
+    t.equal(pool.nodesToHost(nodes, 'http:').length, 1);
+    t.equal(pool.nodesToHost(nodes, 'http:')[0].url.host, '127.0.0.1:9200');
+    t.end();
+  });
+
   t.end();
 });