From efa8f3fedfda7775ffb76fd44ffd8afb32eae857 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?V=C3=ADctor=20Losada=20Hern=C3=A1ndez?= Date: Mon, 16 Dec 2024 22:13:14 +0100 Subject: [PATCH] remove unnecessary, cluttering logs --- server/app.js | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/server/app.js b/server/app.js index c54f601eb..c91bbc57f 100644 --- a/server/app.js +++ b/server/app.js @@ -91,7 +91,6 @@ app.use(cors(corsOptions)); //Account Middleware app.use((req, res, next)=>{ - console.log('passing through acc middleware, checking for cookies now: does cookies exist? ', !!req.cookies, ', ok, does the session cookie exist? ', !!req.cookies.nc_session); if(req.cookies && req.cookies.nc_session){ try { req.account = jwt.decode(req.cookies.nc_session, config.get('secret')); @@ -352,12 +351,8 @@ app.get('/user/:username', async (req, res, next)=>{ //Change author name on brews app.put('/api/user/rename', async (req, res)=>{ const { username, newUsername } = req.body; - - //this next logs will be removed in a next PR, as i need to get this live to test if req.account is created when passing the request from naturalcrit.com - - console.log(`is user ${req.account.username} equal to ${username}? ${req.account.username === username} ${req.account.username === username && 'then add the damn auth for renaming!'}`); - console.log('renaming'); - + console.log(req.account); + if(!username || !newUsername) { return res.status(400).json({ error: 'Username and newUsername are required.' }); }