From ac5a25d7aea0626aa0f42d6bc31399aee6e7e90f Mon Sep 17 00:00:00 2001
From: gumi <git@gumi.ca>
Date: Wed, 11 Apr 2018 12:41:03 -0400
Subject: make the console less verbose

---
 src/routers/tmwa/index.js               | 8 +++-----
 src/routers/tmwa/middlewares/account.js | 8 +++-----
 2 files changed, 6 insertions(+), 10 deletions(-)

(limited to 'src/routers')

diff --git a/src/routers/tmwa/index.js b/src/routers/tmwa/index.js
index 7e1cefe..495240b 100644
--- a/src/routers/tmwa/index.js
+++ b/src/routers/tmwa/index.js
@@ -22,10 +22,8 @@ module.exports = exports = class TMWA {
 
         this.router.get("/server", middlewares.server);
 
-        this.router.all("/account", rate_limit); // filter out the flood
-        this.router.all("/account", challenge); // require a captcha
-        this.router.use("/account", express.json()); // parse the body as json
-        this.router.post("/account", middlewares.account);
+        this.router.all("/account", rate_limit, challenge); // flood limit + captcha
+        this.router.post("/account", express.json(), middlewares.account);
 
         tmwa_poll(this); // first heartbeat
 
@@ -39,7 +37,7 @@ const tmwa_poll = (_this) => {
         const lines = data.split("\n");
 
         if (err || lines.length < 2) {
-            console.error("encountered an error while retrieving online.txt", err);
+            console.error("TMWA: encountered an error while retrieving online.txt", err);
             _this.timeout = setTimeout(() => tmwa_poll(_this), 30000); // <= it failed, so check again later
             return;
         }
diff --git a/src/routers/tmwa/middlewares/account.js b/src/routers/tmwa/middlewares/account.js
index e29af24..1249618 100644
--- a/src/routers/tmwa/middlewares/account.js
+++ b/src/routers/tmwa/middlewares/account.js
@@ -10,7 +10,6 @@ module.exports = exports = (req, res, next) => {
             status: "error",
             error: "malformed request"
         });
-        console.info("a malformed request was received", req.ip, req.body);
         req.app.locals.rate_limiting.add(req.ip);
         setTimeout(() => req.app.locals.rate_limiting.delete(req.ip), 300000);
         return;
@@ -22,7 +21,7 @@ module.exports = exports = (req, res, next) => {
                 status: "error",
                 error: "couldn't reach the database"
             });
-            console.warn("a connection with the database couldn't be established");
+            console.warn("TMWA.account: a connection with the database couldn't be established");
             return;
         }
 
@@ -40,7 +39,6 @@ module.exports = exports = (req, res, next) => {
                         status: "error",
                         error: "already exists"
                     });
-                    console.info("a request to create an already-existent account was received", req.ip, query_params.USERNAME);
                     req.app.locals.rate_limiting.add(req.ip);
                     setTimeout(() => req.app.locals.rate_limiting.delete(req.ip), 2000);
                 } else {
@@ -48,13 +46,13 @@ module.exports = exports = (req, res, next) => {
                         status: "error",
                         error: "couldn't add the user"
                     });
-                    console.error("an unexpected sql error occured", err);
+                    console.error("TMWA.account: an unexpected sql error occured: %s", err.code);
                 }
             } else {
                 res.status(201).json({
                     status: "success"
                 });
-                console.info(`an account was created: ${query_params.USERNAME}`);
+                console.info("TMWA.account: an account was created: %s [%s]", query_params.USERNAME, req.ip);
                 req.app.locals.rate_limiting.add(req.ip);
                 setTimeout(() => req.app.locals.rate_limiting.delete(req.ip), 300000);
             }
-- 
cgit v1.2.3-70-g09d2