From a750a53f6ce936b7520443d4d4a9ca0ce2ef313c Mon Sep 17 00:00:00 2001 From: Louis Capitanchik <contact@louiscap.co> Date: Sat, 22 Feb 2020 16:16:59 +0000 Subject: [PATCH] Don't send empty feedback --- src/config/sequelize.js | 1 + src/http/controllers/api/feedback.js | 7 +++++++ 2 files changed, 8 insertions(+) diff --git a/src/config/sequelize.js b/src/config/sequelize.js index 975f561..8ffb736 100644 --- a/src/config/sequelize.js +++ b/src/config/sequelize.js @@ -13,6 +13,7 @@ const common = { paranoid: true, underscored: true, }, + logging: false, ...(config('database', {})), } diff --git a/src/http/controllers/api/feedback.js b/src/http/controllers/api/feedback.js index dcf1ac3..1348602 100644 --- a/src/http/controllers/api/feedback.js +++ b/src/http/controllers/api/feedback.js @@ -49,6 +49,13 @@ function createSlackPayload({ name, email, message = '' }) { exports.send = async ctx => { const { message } = ctx.request.body + + if (!message) { + ctx.status = 204 + ctx.body = null + return + } + const payload = { message } const user = await ctx.services.authService.getUser() if (user) { -- GitLab