josh/automatisch
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #2491 from automatisch/404-errors

Browse Source 
refactor: Do not log error traces for 404 and 422 errors
This commit is contained in:
Ömer Faruk Aydın
2025-05-22 15:35:35 +02:00
committed by GitHub
parent 0700b465de bc0a8014ab
commit 1acd0c7a14
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
packages/backend/src/helpers/error-handler.js
View File

@@ -19,19 +19,26 @@ import {
// eslint-disable-next-line no-unused-vars
const errorHandler = (error, request, response, next) => {
if (error.message === 'Not Found' || error instanceof NotFoundError) {
logger.http(request.method + ' ' + request.url + ' ' + 404);
response.status(404).end();
return;
}
if (notFoundAppError(error)) {
logger.http(request.method + ' ' + request.url + ' ' + 404);
response.status(404).end();
return;
}
if (error instanceof ValidationError) {
logger.http(request.method + ' ' + request.url + ' ' + 422);
renderObjectionError(response, error, 422);
return;
}
if (error instanceof UniqueViolationError) {
renderUniqueViolationError(response, error);
return;
}
if (error instanceof ForeignKeyViolationError) {