fix minor bug: ensure single database connection on server startup (#…

…2240)

* Fix MongoDB URI in loadPlugin function

* Implement connection check before establishing new database connection

* Replace isConnected variable with mongoose.connection.readyState check

* Add check comments to db.ts file

src/config/plugins/loadPlugins.ts

@@ -6,7 +6,7 @@ import mongoose from "mongoose";
 // Only loads plugin data for the time if it's not currently present in the database
 const loadPlugins = async (): Promise<void> => {
   try {
-    await mongoose.connect(process.env.MONGO_URI as string);
+    await mongoose.connect(process.env.MONGO_DB_URL as string);
     logger.info("\x1b[1m\x1b[32m%s\x1b[0m", `Connected to the database`);
     const res = await Plugin.find();
     const databaseTitle = mongoose.connection.db.databaseName;

src/db.ts

@@ -6,6 +6,12 @@ import { checkReplicaSet } from "./utilities/checkReplicaSet";
 let session!: mongoose.ClientSession;
 
 export const connect = async (dbName?: string): Promise<void> => {
+  // firstly we check if a connection to the database already exists.
+  if (mongoose.connection.readyState !== 0) {
+    // if the connection state is not 0, it means a connection already exists so we return.
+    return;
+  }
+  // if no connection exists, we try to establish a new connection.
   try {
     await mongoose.connect(MONGO_DB_URL as string, {
       dbName,
@@ -52,6 +58,10 @@ export const connect = async (dbName?: string): Promise<void> => {
 };
 
 export const disconnect = async (): Promise<void> => {
+  if (mongoose.connection.readyState === 0) {
+    logger.warn("No active database connection to disconnect.");
+    return;
+  }
   session?.endSession();
   await mongoose.connection.close();
 };