move winston logger

server/dbInit.js

@@ -1,4 +1,5 @@
 const mysql = require('mysql');
+const logger = require('./winstonLogger');
 
 require('dotenv').config();
 const DB_HOST = process.env.DB_HOST;
@@ -17,10 +18,10 @@ const connection = mysql.createConnection({
 
 connection.connect((err) => {
 if (err) {
-    console.error('Error connecting to the database: ' + err.stack);
+    logger.error('Error connecting to database: ' + err);
     return;
 }
-console.log('Connected to the database as id ' + connection.threadId);
+logger.info('Connected to database ' + DB_NAME + ' on ' + DB_HOST + ':' + DB_PORT + ' as user ' + DB_USER);
 });
 
 module.exports = connection;

server/index.js

@@ -1,26 +1,7 @@
 // this nodejs server will serve a rest api for the frontend to consume
 // everything will be in one file for simplicity
 
-// Winston logger
-const winston = require('winston');
-
-const logger = winston.createLogger({
-  level: 'info',
-  format: winston.format.combine(
-    winston.format.timestamp(),
-    winston.format.json()
-  ),
-  transports: [
-    new winston.transports.File({ filename: 'error.log', level: 'error' }),
-    new winston.transports.File({ filename: 'combined.log' }),
-  ],
-});
-
-if (process.env.NODE_ENV !== 'production') {
-  logger.add(new winston.transports.Console({
-    format: winston.format.simple(),
-  }));
-}
+const logger = require('./winstonLogger');
 
 // Environment variables
 require('dotenv').config();

server/winstonLogger.js

@@ -0,0 +1,24 @@
+// Winston logger
+const winston = require('winston');
+
+const logger = winston.createLogger({
+  level: 'info',
+  format: winston.format.combine(
+    winston.format.timestamp(),
+    winston.format.json()
+  ),
+  transports: [
+    new winston.transports.File({ filename: 'error.log', level: 'error' }),
+    new winston.transports.File({ filename: 'combined.log' }),
+  ],
+});
+
+
+if (process.env.NODE_ENV !== 'production') {
+    console.log('Not in production');
+    logger.add(new winston.transports.Console({
+    format: winston.format.simple(),
+    }));
+}
+
+module.exports = logger;