From 2a7005f53c1120714f396b4e4ca72699ba6cf9a0 Mon Sep 17 00:00:00 2001
From: Fredrik Wahlberg <fredrik@wahlberg.se>
Date: Thu, 30 Jan 2025 22:14:25 +0100
Subject: [PATCH] Enhance logging for unauthorized access and task/tag
 operations; remove unused debug statement

---
 middleware/auth.js | 2 +-
 routes/tasks.js    | 2 ++
 server.js          | 2 --
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/middleware/auth.js b/middleware/auth.js
index de23789..48312cb 100644
--- a/middleware/auth.js
+++ b/middleware/auth.js
@@ -6,7 +6,7 @@ const auth = (req, res, next) => {
         return next();
     } else {
         res.status(401).send('Authentication required.');
-        logger.error('Unauthorized access attempted from IP:', req.ip);
+        logger.error(`Unauthorized access attempted from IP: ${req.ip}`);
     }
 };
 
diff --git a/routes/tasks.js b/routes/tasks.js
index 2774af7..026ff9e 100644
--- a/routes/tasks.js
+++ b/routes/tasks.js
@@ -52,6 +52,7 @@ router.post('/add-task', auth, async (req, res) => {
   - State "TODO" from "TODO" [${currentDateTime}]
   :END:
 `;
+        logger.info(`Task added: ${orgFormattedData}`);
     }
 
     const filePath = path.join(dataDir, 'tasks.org');
@@ -71,6 +72,7 @@ router.post('/save-tags', auth, async (req, res) => {
     try {
         await fs.promises.writeFile(filePath, JSON.stringify(tags));
         res.send({ message: 'Tags saved successfully!' });
+        logger.info(`New tags saved: ${tags}`);
     } catch (err) {
         logger.error('Error saving tags:', err);
         res.status(500).send('Error saving tags.');
diff --git a/server.js b/server.js
index f759216..10f791c 100644
--- a/server.js
+++ b/server.js
@@ -4,7 +4,6 @@ const bodyParser = require('body-parser');
 const session = require('express-session');
 const cookieParser = require('cookie-parser');
 const SQLiteStore = require('connect-sqlite3')(session);
-const debug = require('debug')('app');
 const tasksRouter = require('./routes/tasks');
 const authRouter = require('./routes/auth');
 const authMiddleware = require('./middleware/auth');
@@ -39,5 +38,4 @@ app.use('/', authMiddleware, tasksRouter);
 
 app.listen(port, () => {
     logger.info(`Server running at http://localhost:${port}`);
-    //debug(`Server running at http://localhost:${port}`);
 });