From 0c2c060acba9e8e2e096a964ffcad9b0c98cc42c Mon Sep 17 00:00:00 2001 From: Khaled Garbaya Date: Wed, 4 Oct 2017 10:59:06 +0200 Subject: [PATCH] feat(handlers): Add error handling for settings page --- routes/settings.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/routes/settings.js b/routes/settings.js index bf8cfbb..dc527b4 100644 --- a/routes/settings.js +++ b/routes/settings.js @@ -1,18 +1,19 @@ const express = require('express') +const { catchErrors } = require('../handlers/errorHandlers') const router = express.Router() /* GET settings page. */ -router.get('/', function (req, res, next) { +router.get('/', catchErrors(async function (req, res, next) { const cookie = req.cookies.theExampleAppSettings const settings = cookie || { cpa: '', cda: '', space: '' } res.render('settings', { title: 'Settings', settings }) -}) +})) /* POST settings page. */ -router.post('/', function (req, res, next) { +router.post('/', catchErrors(async function (req, res, next) { const settings = {space: req.body.space, cda: req.body.cda, cpa: req.body.cpa} res.cookie('theExampleAppSettings', settings, { maxAge: 900000, httpOnly: true }) res.render('settings', settings) -}) +})) module.exports = router