Merge pull request Automattic#15089 from Automattic/vkarpov15/Automatticgh-15042

fix(connection): remove heartbeat check in load balanced mode

Author: vkarpov15

lib/connection.js

@@ -111,6 +111,9 @@ Object.defineProperty(Connection.prototype, 'readyState', {
     if (
       this._readyState === STATES.connected &&
       this._lastHeartbeatAt != null &&
+      // LoadBalanced topology (behind haproxy, including Atlas serverless instances) don't use heartbeats,
+      // so we can't use this check in that case.
+      this.client?.topology?.s?.description?.type !== 'LoadBalanced' &&
       typeof this.client?.topology?.s?.description?.heartbeatFrequencyMS === 'number' &&
       Date.now() - this._lastHeartbeatAt >= this.client.topology.s.description.heartbeatFrequencyMS * 2) {
       return STATES.disconnected;

lib/drivers/node-mongodb-native/connection.js

@@ -426,9 +426,6 @@ function _setClient(conn, client, options, dbName) {
   }
 
   conn.onOpen();
-  if (client.topology?.s?.state === 'connected') {
-    conn._lastHeartbeatAt = Date.now();
-  }
 
   for (const i in conn.collections) {
     if (utils.object.hasOwnProperty(conn.collections, i)) {