From a9363620515d25bdb52770eb84f790d435a7a783 Mon Sep 17 00:00:00 2001 From: Nazar Gargol Date: Tue, 11 Jun 2019 12:35:16 +0200 Subject: [PATCH] Removed logging for validation error in frontend routing refs #10525 - The logging in this place only introduced confusion and provided no usefull information --- core/server/services/routing/helpers/error.js | 4 ---- 1 file changed, 4 deletions(-) diff --git a/core/server/services/routing/helpers/error.js b/core/server/services/routing/helpers/error.js index ce658479fd..dfe3edc509 100644 --- a/core/server/services/routing/helpers/error.js +++ b/core/server/services/routing/helpers/error.js @@ -1,5 +1,3 @@ -const common = require('../../../lib/common'); - /** * @description Centralised error handling for API requests. * @param {Function} next @@ -16,8 +14,6 @@ function handleError(next) { // this post from Content API (by slug), but this is not a valid slug. With dynamic routing we cannot // add a regex to the target express route, because we don't know if people use /:slug/ or not. It's dynamic. if (err.errorType === 'ValidationError') { - common.logging.error(err); - // @NOTE: Just try next router, it will end in a 404 if no router can resolve the request. return next(); }