summary refs log tree commit diff
path: root/api/src
diff options
context:
space:
mode:
authorFlam3rboy <34555296+Flam3rboy@users.noreply.github.com>2021-09-03 04:04:34 +0200
committerFlam3rboy <34555296+Flam3rboy@users.noreply.github.com>2021-09-03 04:04:34 +0200
commit11b2176773fbfed96d06d33075baf0785daaf69b (patch)
treee5de3fe2be7d0cd1cf4c22e8d062fb32f7c30830 /api/src
parent:bug: fix #289 (diff)
downloadserver-11b2176773fbfed96d06d33075baf0785daaf69b.tar.xz
:bug: fix message sending
Diffstat (limited to 'api/src')
-rw-r--r--api/src/middlewares/ErrorHandler.ts9
-rw-r--r--api/src/routes/channels/#channel_id/messages/#message_id/index.ts5
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)
 	]);