Refactor logging implementation and enhance authentication logging

logger.js

@@ -4,11 +4,11 @@ const logger = createLogger({
     level: 'info',
     format: format.combine(
         format.timestamp(),
-        format.json()
+        format.printf(info => `${info.timestamp} ${info.level}: ${info.message}`)
     ),
     transports: [
         new transports.Console(),
-        new transports.File({ filename: 'app.log' })
+        new transports.File({ filename: '/data/app.log' })
     ]
 }); 
 

middleware/auth.js

@@ -1,10 +1,12 @@
 const basicAuth = require('basic-auth');
+const logger = require('../logger');
 
 const auth = (req, res, next) => {
     if (req.session && req.session.user) {
         return next();
     } else {
         res.status(401).send('Authentication required.');
+        logger.error('Unauthorized access attempted from IP:', req.ip);
     }
 };
 

public/js/main.js

@@ -2,7 +2,9 @@ import { checkSession, login, logout } from './auth.js';
 import { saveTask } from './tasks.js';
 import { saveTags, loadTags } from './tags.js';
 
-navigator.serviceWorker.register('/service-worker.js')
+if ('serviceWorker' in navigator) {
+    window.addEventListener('load', () => {
+        navigator.serviceWorker.register('/service-worker.js')
             .then(registration => {
                 console.log('ServiceWorker registration successful with scope: ', registration.scope);
 
@@ -27,6 +29,9 @@ navigator.serviceWorker.register('/service-worker.js')
             .catch(error => {
                 console.log('ServiceWorker registration failed: ', error);
             });
+    }); 
+}   
+
 
 document.addEventListener('DOMContentLoaded', function() {
     const loginForm = document.getElementById('loginForm');

routes/auth.js

@@ -1,5 +1,6 @@
 const express = require('express');
 const basicAuth = require('basic-auth');
+const logger = require('../logger');
 const router = express.Router();
 
 router.post('/login', (req, res) => {
@@ -10,6 +11,7 @@ router.post('/login', (req, res) => {
     if (user && user.name === username && user.pass === password) {
         req.session.user = user.name;
         res.status(200).send('Login successful');
+        logger.info(`User ${user.name} logged in`); 
     } else {
         res.status(401).send('Authentication required');
     }

server.js

@@ -8,6 +8,7 @@ const debug = require('debug')('app');
 const tasksRouter = require('./routes/tasks');
 const authRouter = require('./routes/auth');
 const authMiddleware = require('./middleware/auth');
+const logger = require('./logger');
 
 const app = express();
 const port = 3044;
@@ -37,5 +38,6 @@ app.use('/', authRouter);
 app.use('/', authMiddleware, tasksRouter);
 
 app.listen(port, () => {
-    debug(`Server running at http://localhost:${port}`);
+    logger.info(`Server running at http://localhost:${port}`);
+    //debug(`Server running at http://localhost:${port}`);
 });