diff options
author | Flam3rboy <34555296+Flam3rboy@users.noreply.github.com> | 2021-09-03 04:04:34 +0200 |
---|---|---|
committer | Flam3rboy <34555296+Flam3rboy@users.noreply.github.com> | 2021-09-03 04:04:34 +0200 |
commit | 11b2176773fbfed96d06d33075baf0785daaf69b (patch) | |
tree | e5de3fe2be7d0cd1cf4c22e8d062fb32f7c30830 /api/src | |
parent | :bug: fix #289 (diff) | |
download | server-11b2176773fbfed96d06d33075baf0785daaf69b.tar.xz |
:bug: fix message sending
Diffstat (limited to 'api/src')
-rw-r--r-- | api/src/middlewares/ErrorHandler.ts | 9 | ||||
-rw-r--r-- | api/src/routes/channels/#channel_id/messages/#message_id/index.ts | 5 |
2 files changed, 7 insertions, 7 deletions
diff --git a/api/src/middlewares/ErrorHandler.ts b/api/src/middlewares/ErrorHandler.ts index f061172a..34827cab 100644 --- a/api/src/middlewares/ErrorHandler.ts +++ b/api/src/middlewares/ErrorHandler.ts @@ -2,7 +2,7 @@ import { NextFunction, Request, Response } from "express"; import { HTTPError } from "lambert-server"; import { EntityNotFoundError } from "typeorm"; import { FieldError } from "../util/instanceOf"; -import {ApiError} from "../util/ApiError"; +import { ApiError } from "../util/ApiError"; // TODO: update with new body/typorm validation export function ErrorHandler(error: Error, req: Request, res: Response, next: NextFunction) { @@ -19,16 +19,15 @@ export function ErrorHandler(error: Error, req: Request, res: Response, next: Ne code = error.code; message = error.message; httpcode = error.httpStatus; - } - else if (error instanceof EntityNotFoundError) { - message = `${(error as any).stringifyTarget} can not be found`; + } else if (error as EntityNotFoundError) { + message = `${(error as any).stringifyTarget} could not be found`; code = 404; } else if (error instanceof FieldError) { code = Number(error.code); message = error.message; errors = error.errors; } else { - console.error(`[Error] ${code} ${req.url}`, errors || error, "body:", req.body); + console.error(`[Error] ${code} ${req.url}\n`, errors || error, "\nbody:", req.body); if (req.server?.options?.production) { // don't expose internal errors to the user, instead human errors should be thrown as HTTPError diff --git a/api/src/routes/channels/#channel_id/messages/#message_id/index.ts b/api/src/routes/channels/#channel_id/messages/#message_id/index.ts index 9c631a18..7a00de43 100644 --- a/api/src/routes/channels/#channel_id/messages/#message_id/index.ts +++ b/api/src/routes/channels/#channel_id/messages/#message_id/index.ts @@ -20,7 +20,8 @@ router.patch("/", check(MessageCreateSchema), async (req: Request, res: Response } const new_message = await handleMessage({ - // TODO: should be message_reference overridable? + ...message, + // TODO: should message_reference be overridable? // @ts-ignore message_reference: message.message_reference, ...body, @@ -35,7 +36,7 @@ router.patch("/", check(MessageCreateSchema), async (req: Request, res: Response await emitEvent({ event: "MESSAGE_UPDATE", channel_id, - data: { ...message, nonce: undefined } + data: { ...new_message, nonce: undefined } } as MessageUpdateEvent) ]); |