diff --git a/server.ts b/server.ts index eccd19919b3d56db1009f31ed7aa2626a6a3f665..2db39ab06c3c0813d4449f4ee3ba1219cbae825f 100644 --- a/server.ts +++ b/server.ts @@ -166,7 +166,10 @@ app.use(function (err, req, res, next) { error = err.stack || err.message || err } - logger.error('Error in controller.', { err: error }) + // Sequelize error + const sql = err.parent ? err.parent.sql : undefined + + logger.error('Error in controller.', { err: error, sql }) return res.status(err.status || 500).end() }) diff --git a/server/helpers/logger.ts b/server/helpers/logger.ts index ce6e38f158f5df297a0083dbc6692b69229bc0b3..e1e3a1fe20f9dbea9f6a3fb7f0c623fc6a4056bd 100644 --- a/server/helpers/logger.ts +++ b/server/helpers/logger.ts @@ -22,7 +22,13 @@ function loggerReplacer (key: string, value: any) { } const consoleLoggerFormat = winston.format.printf(info => { - let additionalInfos = JSON.stringify(info.meta || info.err, loggerReplacer, 2) + const obj = { + meta: info.meta, + err: info.err, + sql: info.sql + } + + let additionalInfos = JSON.stringify(obj, loggerReplacer, 2) if (additionalInfos === undefined || additionalInfos === '{}') additionalInfos = '' else additionalInfos = ' ' + additionalInfos diff --git a/server/middlewares/cache.ts b/server/middlewares/cache.ts index b40486e4bf3b953f83e907d500e4c8117f8a40b9..1b44957d3004ac8106ea30df3620d5150e35eb53 100644 --- a/server/middlewares/cache.ts +++ b/server/middlewares/cache.ts @@ -31,6 +31,8 @@ function cacheRoute (lifetimeArg: string | number) { logger.error('Cannot cache route.', { err }) return done(err) }) + } else { + done() } return sendSave(body) @@ -52,7 +54,7 @@ function cacheRoute (lifetimeArg: string | number) { return done() }) } catch (err) { - logger.error('Cannot serve cached route.', err) + logger.error('Cannot serve cached route.', { err }) return next() } }