diff --git a/lib/middlewares/authorize.js b/lib/middlewares/authorize.js index d8781882..a76a7438 100644 --- a/lib/middlewares/authorize.js +++ b/lib/middlewares/authorize.js @@ -1,19 +1,21 @@ const pairing = require('../pairing') +const logger = require('../logger') const authorize = function (req, res, next) { - const deviceId = req.deviceId - - return pairing.isPaired(deviceId) + return pairing.isPaired(req.deviceId) .then(deviceName => { if (deviceName) { - req.deviceId = deviceId req.deviceName = deviceName return next() } + logger.error(`Device ${req.deviceId} not found`) return res.status(403).json({ error: 'Forbidden' }) }) - .catch(next) + .catch(error => { + logger.error(error) + return next() + }) } module.exports = authorize diff --git a/lib/middlewares/ca.js b/lib/middlewares/ca.js index a43af019..0c202d0a 100644 --- a/lib/middlewares/ca.js +++ b/lib/middlewares/ca.js @@ -1,11 +1,15 @@ const pairing = require('../pairing') +const logger = require('../logger') function ca (req, res) { const token = req.query.token return pairing.authorizeCaDownload(token) .then(ca => res.json({ ca })) - .catch(() => res.status(403).json({ error: 'forbidden' })) + .catch(error => { + logger.error(error.message) + res.status(403).json({ error: 'forbidden' }) + }) } module.exports = ca