Merge pull request #209 from gnmyt/fixes/server-bugs

🐛 Wichtige Bug-Fixes im Server

server/index.js

@@ -14,6 +14,7 @@ process.on('uncaughtException', err => require('./config/errorHandler')(err));
 
 // Register middlewares
 app.use(express.json());
+app.use(require('./middlewares/error'));
 
 // Register routes
 app.use("/api/config", require('./routes/config'));

server/middlewares/error.js

@@ -0,0 +1,5 @@
+module.exports = (err, req, res, next) => {
+    if (!(err instanceof SyntaxError)) return next();
+
+    res.status(400).json({message: "You need to provide a valid JSON body"});
+}

server/routes/config.js

@@ -19,7 +19,7 @@ app.get("/", password(true), async (req, res) => {
 
 // Updates a specific config entry
 app.patch("/:key", password(false), async (req, res) => {
-    if (!req.body.value.toString()) return res.status(400).json({message: "You need to provide the new value"});
+    if (!req.body.value?.toString()) return res.status(400).json({message: "You need to provide the new value"});
 
     if ((req.params.key === "ping" || req.params.key === "download" || req.params.key === "upload") && isNaN(req.body.value))
         return res.status(400).json({message: "You need to provide a number in order to change this"});