chore: add logic to close dB connections after it's been used

backend/src/connectToDB.ts

@@ -19,5 +19,13 @@ async function connectToDB(): Promise<typeof mongoose> {
     throw error
   }
 }
+async function closeDBConnection(): Promise<void> {
+  try {
+    await mongoose.connection.close()
+    console.log('Disconnected successfully from MongoDB server')
+  } catch (error) {
+    console.error('Error closing the database connection:', error)
+  }
+}
 
-export { connectToDB }
+export { connectToDB, closeDBConnection }

backend/src/server.ts

@@ -3,7 +3,7 @@ import jwt from 'jsonwebtoken'
 import cors from 'cors'
 import bcrypt from 'bcrypt'
 import dotenv from 'dotenv'
-import { connectToDB } from './connectToDB'
+import { connectToDB, closeDBConnection } from './connectToDB'
 import User from './models/User'
 
 export const app = express()
@@ -66,6 +66,8 @@ app.post('/api/v1/auth/signup', async (req, res) => {
   } catch (error) {
     console.error('Signup error:', error)
     res.status(500).send('Server error')
+  } finally {
+    await closeDBConnection()
   }
 })
 
@@ -95,6 +97,8 @@ app.post('/api/v1/auth/login', async (req, res) => {
   } catch (error) {
     console.error('Login error:', error)
     res.status(500).send('Server error')
+  } finally {
+    await closeDBConnection()
   }
 })
 
@@ -103,13 +107,16 @@ app.delete('/api/v1/auth/delete', async (req, res) => {
   const { username } = req.body
 
   try {
+    await connectToDB()
     const user = await User.findOneAndDelete({ username })
     if (!user) {
       return res.status(404).send('User not found')
     }
     res.status(200).send('User deleted successfully')
   } catch (error) {
     res.status(500).send('Internal server error')
+  } finally {
+    await closeDBConnection()
   }
 })