fix: Resolve TypeError by correcting module dependency injection
This commit fixes a TypeError that occurred because the contactRoutes module was being initialized before the contactController module. The fix involves: - Swapping the initialization order in server.js so the contactController is initialized before contactRoutes. - Modifying contactRoutes.js to accept the initialized contactController as a dependency, rather than requiring it on its own.
This commit is contained in:
@@ -1,8 +1,11 @@
|
|||||||
|
// The entire module is now a function that accepts 'contactController' as an argument.
|
||||||
|
module.exports = (contactController) => {
|
||||||
const express = require('express');
|
const express = require('express');
|
||||||
const router = express.Router();
|
const router = express.Router();
|
||||||
const rateLimit = require('express-rate-limit');
|
const rateLimit = require('express-rate-limit');
|
||||||
const { body, validationResult } = require('express-validator');
|
const { body, validationResult } = require('express-validator');
|
||||||
const contactController = require('../controllers/contactController');
|
// Remove the old require statement since we are getting the controller as an argument
|
||||||
|
// const contactController = require('../controllers/contactController');
|
||||||
const { formSecurityCheck } = require('../middleware/securityMw');
|
const { formSecurityCheck } = require('../middleware/securityMw');
|
||||||
|
|
||||||
// 🛡️ Configure rate limiting to prevent DDoS and spamming
|
// 🛡️ Configure rate limiting to prevent DDoS and spamming
|
||||||
@@ -39,4 +42,6 @@ router.post('/submit-form',
|
|||||||
contactController.submitForm
|
contactController.submitForm
|
||||||
);
|
);
|
||||||
|
|
||||||
module.exports = router;
|
// Return the configured router
|
||||||
|
return router;
|
||||||
|
};
|
@@ -34,10 +34,11 @@ const transporter = nodemailer.createTransport({
|
|||||||
},
|
},
|
||||||
});
|
});
|
||||||
|
|
||||||
// Import contactRoutes and contactController
|
|
||||||
const contactRoutes = require('./routes/contactRoutes');
|
|
||||||
const contactController = require('./controllers/contactController')(pool, transporter);
|
const contactController = require('./controllers/contactController')(pool, transporter);
|
||||||
|
|
||||||
|
// Import contactRoutes and contactController
|
||||||
|
const contactRoutes = require('./routes/contactRoutes')(contactController);
|
||||||
|
|
||||||
// Use contactRoutes to connect the modular router to the main app
|
// Use contactRoutes to connect the modular router to the main app
|
||||||
app.use(contactRoutes);
|
app.use(contactRoutes);
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user