Skip to content

Commit 791627b

Browse files
committed
fix(errors): errors not being printed when wrapped with json()
1 parent 933b81c commit 791627b

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

src/controllers/data.controller.ts

+2-4
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ export async function extractInfo(
154154
// Then run the topic extractor
155155
const userTopics = extractUserKeywords(
156156
text,
157-
JSON.parse(request.body['keywords'] || {}),
157+
JSON.parse(request.body.keywords || '{}'),
158158
)
159159
// Add the topics to the list
160160
for (const topic of userTopics) {
@@ -175,9 +175,7 @@ export async function extractInfo(
175175
},
176176
})
177177
} catch (error) {
178-
Logger.error(
179-
`controller.data.extractInfo: caught error: ${json(error)}`,
180-
)
178+
Logger.error(`controller.data.extractInfo: caught error: ${error}`)
181179
// If an error is caught, forward it to the error handler
182180
next(error)
183181
}

src/utils/errors.util.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,8 @@ export default function errorHandler(
9999
})
100100
} else {
101101
// Log it
102-
Logger.error(`middleware.error: server crash - ${json(error)}`)
102+
Logger.error(`middleware.error: server crash - ${error}`)
103+
console.error(error)
103104
// Else the server has crashed, return an internalServerError
104105
response.status(500).json({
105106
code: 500,

0 commit comments

Comments
 (0)