debug logging and fixes

Author: Unreal-Dan

app.js

@@ -94,6 +94,13 @@ app.use((req, res, next) => {
       console.log('[SESS]', req.method, req.originalUrl,
         'cookie:', req.headers.cookie ? req.headers.cookie.substring(0, 60) : 'MISSING',
         'user:', req.user ? req.user.id : 'none');
+      const origSetHeader = res.setHeader.bind(res);
+      res.setHeader = function(name, value) {
+        if (name.toLowerCase() === 'set-cookie') {
+          console.log('[SET-COOKIE]', value);
+        }
+        return origSetHeader(name, value);
+      };
     }
     next();
 });

routes/mode.js

@@ -12,6 +12,13 @@ const execAsync = util.promisify(exec);
 const spawn = require('child_process').spawn;
 require('dotenv').config();
 
+router.param('modeId', (req, res, next, val) => {
+  if (!/^\d+$/.test(val)) {
+    return res.status(404).render('not-found');
+  }
+  next();
+});
+
 const ledCounts = {
   'Gloves': 10,
   'Orbit': 28,

routes/pat.js

@@ -13,6 +13,20 @@ const spawn = require('child_process').spawn;
 const sequelize = require('../config/database-pg');
 require('dotenv').config();
 
+router.param('patId', (req, res, next, val) => {
+  if (!/^\d+$/.test(val)) {
+    return res.status(404).render('not-found');
+  }
+  next();
+});
+
+router.param('patternId', (req, res, next, val) => {
+  if (!/^\d+$/.test(val)) {
+    return res.status(404).render('not-found');
+  }
+  next();
+});
+
 router.get('/:patId', async (req, res) => {
   try {
     const pat = await PatternSet.findByPk(req.params.patId, {