fix: clear timeout in collection operations to prevent memory leak

lib/connection.js

@@ -69,7 +69,6 @@ function Connection(base) {
   this.config = {};
   this.replica = false;
   this.options = null;
-  this.otherDbs = []; // FIXME: To be replaced with relatedDbs
   this.relatedDbs = {}; // Hashmap of other dbs that share underlying connection
   this.states = STATES;
   this._readyState = STATES.disconnected;
@@ -137,8 +136,8 @@ Object.defineProperty(Connection.prototype, 'readyState', {
 
     if (this._readyState !== val) {
       this._readyState = val;
-      // [legacy] loop over the otherDbs on this connection and change their state
-      for (const db of this.otherDbs) {
+      // loop over the relatedDbs on this connection and change their state
+      for (const db of Object.values(this.relatedDbs)) {
         db.readyState = val;
       }
 
@@ -1330,7 +1329,7 @@ Connection.prototype.onClose = function onClose(force) {
 
   const wasForceClosed = typeof force === 'object' && force !== null ? force.force : force;
 
-  for (const db of this.otherDbs) {
+  for (const db of Object.values(this.relatedDbs)) {
     this._destroyCalled ? db.destroy({ force: wasForceClosed, skipCloseClient: true }) : db.close({ force: wasForceClosed, skipCloseClient: true });
   }
 };

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

@@ -239,6 +239,9 @@
 
       if (syncCollectionMethods[i] && typeof lastArg === 'function') {
         const result = collection[i].apply(collection, _args.slice(0, _args.length - 1));
+        if (timeout != null) {
+          clearTimeout(timeout);
+        }
         this.conn.emit('operation-end', { _id: opId, modelName: _this.modelName, collectionName: this.name, method: i, result });
         return lastArg.call(this, null, result);
       }
@@ -247,6 +250,9 @@
       if (ret != null && typeof ret.then === 'function') {
         return ret.then(
           result => {
+            if (timeout != null) {
+              clearTimeout(timeout);
+            }
             if (typeof lastArg === 'function') {
               lastArg(null, result);
             } else {
@@ -255,6 +261,9 @@
             return result;
           },
           error => {
+            if (timeout != null) {
+              clearTimeout(timeout);
+            }
             if (typeof lastArg === 'function') {
               lastArg(error);
               return;
@@ -265,10 +274,16 @@
           }
         );
       }
+      if (timeout != null) {
+        clearTimeout(timeout);
+      }
       return ret;
     } catch (error) {
       // Collection operation may throw because of max bson size, catch it here
       // See gh-3906
+      if (timeout != null) {
+        clearTimeout(timeout);
+      }
       if (typeof lastArg === 'function') {
         return lastArg(error);
       } else {

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

@@ -113,14 +113,13 @@ NativeConnection.prototype.useDb = function(name, options) {
 
   newConn.name = name;
 
-  // push onto the otherDbs stack, this is used when state changes
-  this.otherDbs.push(newConn);
-  newConn.otherDbs.push(this);
-
-  // push onto the relatedDbs cache, this is used when state changes
-  if (options && options.useCache) {
-    this.relatedDbs[newConn.name] = newConn;
-    newConn.relatedDbs = this.relatedDbs;
+  // Add to relatedDbs for state synchronization and caching
+  this.relatedDbs[newConn.name] = newConn;
+  newConn.relatedDbs = this.relatedDbs;
+
+  // Ensure the parent connection is also tracked in the new connection's relatedDbs
+  if (this.name && !newConn.relatedDbs[this.name]) {
+    newConn.relatedDbs[this.name] = this;
   }
 
   return newConn;
@@ -160,19 +159,21 @@ NativeConnection.prototype.aggregate = function aggregate(pipeline, options) {
  */
 
 NativeConnection.prototype.removeDb = function removeDb(name) {
-  const dbs = this.otherDbs.filter(db => db.name === name);
-  if (!dbs.length) {
+  const db = this.relatedDbs[name];
+  if (!db) {
     throw new MongooseError(`No connections to database "${name}" found`);
   }
 
-  for (const db of dbs) {
-    db._closeCalled = true;
-    db._destroyCalled = true;
-    db._readyState = STATES.disconnected;
-    db.$wasForceClosed = true;
+  db._closeCalled = true;
+  db._destroyCalled = true;
+  db._readyState = STATES.disconnected;
+  db.$wasForceClosed = true;
+
+  // Remove from all related connections' relatedDbs
+  for (const relatedDb of Object.values(this.relatedDbs)) {
+    delete relatedDb.relatedDbs[name];
   }
   delete this.relatedDbs[name];
-  this.otherDbs = this.otherDbs.filter(db => db.name !== name);
 };
 
 /**
@@ -345,8 +346,10 @@ NativeConnection.prototype.createClient = async function createClient(uri, optio
 
   _setClient(this, client, options, dbName);
 
-  for (const db of this.otherDbs) {
-    _setClient(db, client, {}, db.name);
+  for (const db of Object.values(this.relatedDbs)) {
+    if (db !== this) {
+      _setClient(db, client, {}, db.name);
+    }
   }
   return this;
 };
@@ -494,8 +497,10 @@ function _setClient(conn, client, options, dbName) {
 
   client.on('serverHeartbeatSucceeded', () => {
     conn._lastHeartbeatAt = Date.now();
-    for (const otherDb of conn.otherDbs) {
-      otherDb._lastHeartbeatAt = conn._lastHeartbeatAt;
+    for (const otherDb of Object.values(conn.relatedDbs)) {
+      if (otherDb !== conn) {
+        otherDb._lastHeartbeatAt = conn._lastHeartbeatAt;
+      }
     }
   });
 

test/collection.timeout.test.js

@@ -0,0 +1,57 @@
+'use strict';
+
+const assert = require('assert');
+const mongoose = require('../index');
+
+describe('Collection timeout cleanup', function() {
+  let db;
+
+  before(async function() {
+    db = await mongoose.createConnection('mongodb://127.0.0.1:27017/mongoose_test').asPromise();
+  });
+
+  after(async function() {
+    await db.close();
+  });
+
+  it('should clear timeout on successful operations', async function() {
+    const TestModel = db.model('Test', { name: String });
+
+    // Track active handles before operations
+    const initialHandles = process._getActiveHandles().length;
+
+    // Execute multiple operations that should complete successfully
+    for (let i = 0; i < 10; i++) {
+      await TestModel.find({}).exec();
+    }
+
+    // Allow some time for any lingering timeouts
+    await new Promise(resolve => setTimeout(resolve, 100));
+
+    // Active handles should not have grown significantly
+    const finalHandles = process._getActiveHandles().length;
+    assert.ok(finalHandles <= initialHandles + 2, 
+      `Expected handles to remain stable, but grew from ${initialHandles} to ${finalHandles}`);
+  });
+
+  it('should clear timeout on operation errors', async function() {
+    const TestModel = db.model('Test2', { name: String });
+
+    const initialHandles = process._getActiveHandles().length;
+
+    // Execute operations that will fail
+    for (let i = 0; i < 5; i++) {
+      try {
+        await TestModel.collection.findOne({ $invalidOperator: true });
+      } catch (err) {
+        // Expected to fail
+      }
+    }
+
+    await new Promise(resolve => setTimeout(resolve, 100));
+
+    const finalHandles = process._getActiveHandles().length;
+    assert.ok(finalHandles <= initialHandles + 2,
+      `Expected handles to remain stable after errors, but grew from ${initialHandles} to ${finalHandles}`);
+  });
+});