summary refs log tree commit diff
diff options
context:
space:
mode:
-rw-r--r--src/api/Server.ts7
1 files changed, 4 insertions, 3 deletions
diff --git a/src/api/Server.ts b/src/api/Server.ts
index 6c4b5652..c7e53d70 100644
--- a/src/api/Server.ts
+++ b/src/api/Server.ts
@@ -74,15 +74,16 @@ export class FosscordServer extends Server {
 			path.join(__dirname, "routes", "/"),
 		);
 
+		// 404 is not an error in express, so this should not be an error middleware
+		// this is a fine place to put the 404 handler because its after we register the routes
+		// and since its not an error middleware, our error handler below still works.
 		api.use(
 			"*",
-			(error: any, req: Request, res: Response, next: NextFunction) => {
-				if (error) return next(error);
+			(req: Request, res: Response, next: NextFunction) => {
 				res.status(404).json({
 					message: "404 endpoint not found",
 					code: 0,
 				});
-				next();
 			},
 		);