removed unwanted logger ✔

Author: sank2000

src/controllers/admin/admin.js

@@ -6,7 +6,6 @@ import config from 'config';
 import bcrypt from 'bcrypt';
 
 import { Section, Student, Staff, validateAdmin, Admin } from 'models';
-import logger from 'tools/logging';
 
 /**
  *
@@ -21,7 +20,6 @@ import logger from 'tools/logging';
 
 export const newAdmin = async (req, res) => {
 	const { body } = req;
-	logger.debug('Acknowledged: ', body);
 
 	const { error } = validateAdmin(body);
 	if (error) return res.status(StatusCodes.BAD_REQUEST).json({ error: error.details[0].message });
@@ -89,7 +87,6 @@ export const deleteAdmin = async (req, res) => {
 	const {
 		body: { id }
 	} = req;
-	logger.debug('Acknowledged: ', id);
 
 	if (!id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 

src/controllers/admin/section.js

@@ -17,7 +17,6 @@ import logger from 'tools/logging';
 
 export const newSection = async (req, res) => {
 	const { body } = req;
-	logger.info('Acknowledged: ', body);
 
 	const { error } = validateSection(body);
 	if (error) return res.status(StatusCodes.BAD_REQUEST).json({ error: error.details[0].message });
@@ -90,7 +89,6 @@ export const updateSectionName = async (req, res) => {
 	const {
 		body: { id, name }
 	} = req;
-	logger.debug('Acknowledged: ', id);
 
 	if (!id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 	if (!name) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'name field required' });
@@ -124,7 +122,6 @@ export const updateSectionStaff = async (req, res) => {
 	const {
 		body: { id, staffId }
 	} = req;
-	logger.debug('Acknowledged: ', id);
 
 	if (!id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 	if (!staffId)
@@ -175,7 +172,6 @@ export const updateSectionStudent = async (req, res) => {
 	const {
 		body: { id, studentId }
 	} = req;
-	logger.debug('Acknowledged: ', id);
 
 	if (!id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 	if (!studentId)
@@ -234,7 +230,6 @@ export const removeSectionStaff = async (req, res) => {
 	const {
 		body: { id, staffId }
 	} = req;
-	logger.debug('Acknowledged: ', id);
 
 	if (!id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 	if (!staffId)
@@ -285,7 +280,6 @@ export const removeSectionStudent = async (req, res) => {
 	const {
 		body: { id, studentId }
 	} = req;
-	logger.debug('Acknowledged: ', id);
 
 	if (!id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 	if (!studentId)
@@ -331,7 +325,6 @@ export const deleteSection = async (req, res) => {
 	const {
 		body: { id }
 	} = req;
-	logger.debug('Acknowledged: ', id);
 
 	if (!id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 

src/controllers/admin/staff.js

@@ -18,7 +18,6 @@ import logger from 'tools/logging';
 
 export const newStaff = async (req, res) => {
 	const { body } = req;
-	logger.debug('Acknowledged: ', body);
 
 	const { error } = validateStaff(body);
 	if (error) return res.status(StatusCodes.BAD_REQUEST).json({ error: error.details[0].message });
@@ -100,7 +99,6 @@ export const getStaffById = async (req, res) => {
 
 export const updateStaff = async (req, res) => {
 	const { body } = req;
-	logger.debug('Acknowledged: ', body.id);
 
 	if (!body.id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 
@@ -134,7 +132,6 @@ export const deleteStaff = async (req, res) => {
 	const {
 		body: { id }
 	} = req;
-	logger.debug('Acknowledged: ', id);
 
 	if (!id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 

src/controllers/admin/student.js

@@ -17,7 +17,6 @@ import logger from 'tools/logging';
 
 export const newStudent = async (req, res) => {
 	const { body } = req;
-	logger.debug('Acknowledged: ', body);
 
 	const { error } = validateStudent(body);
 	if (error) return res.status(StatusCodes.BAD_REQUEST).json({ error: error.details[0].message });
@@ -125,7 +124,6 @@ export const getStudentByRegisterNumber = async (req, res) => {
 
 export const updateStudent = async (req, res) => {
 	const { body } = req;
-	logger.debug('Acknowledged: ', body.id);
 
 	if (!body.id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 
@@ -160,7 +158,6 @@ export const deleteStudent = async (req, res) => {
 	const {
 		body: { id }
 	} = req;
-	logger.debug('Acknowledged: ', id);
 
 	if (!id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 

src/controllers/staff.js

@@ -21,7 +21,6 @@ import { Student, Staff, Certificate } from 'models';
  */
 export const getStudentById = async (req, res) => {
 	const { id } = req.params;
-	logger.debug('Acknowleged:', id);
 
 	if (!id) return res.status(StatusCodes.BAD_REQUEST).json({ error: 'id field required' });
 
@@ -103,7 +102,6 @@ export const updateStaff = async (req, res) => {
 	const {
 		body: { phoneNumber, email }
 	} = req;
-	logger.debug('Acknowledged: ', id, ' ', phoneNumber, ' ', email);
 
 	let fields = {
 		phoneNumber,

src/dbConnection.js

@@ -16,10 +16,11 @@ export const dbConnection = async () => {
 	};
 	try {
 		logger.debug(`Database server is at ${chalk.cyan(DB_SERVER)}`);
-		mongoose.connect(DB_SERVER, options);
-		await logger.info(`Database connection ${chalk.greenBright('successful')}`);
+		await mongoose.connect(DB_SERVER, options);
+		logger.info(`Database connection ${chalk.greenBright('successful')}`);
 	} catch (err) {
 		logger.error(`Database connection ${chalk.redBright('failed.')}`);
-		logger.error(`Error: could not connect to the database at ${DB_SERVER}\n`, err);
+		logger.error(`Error: could not connect to the database at ${DB_SERVER}\n`);
+		logger.error(err);
 	}
 };

src/middlewares/errorHandler.js

@@ -4,7 +4,7 @@ import { StatusCodes } from 'http-status-codes';
 export default (err, req, res, next) => {
 	const { body } = req;
 
-	logger.error(err.message, err);
+	logger.error(err.message ?? err);
 
 	if (err.code === 11000) {
 		const errorKeys = Object.keys(err.keyPattern);