diff options
author | Madeline <46743919+MaddyUnderStars@users.noreply.github.com> | 2022-11-05 21:59:12 +1100 |
---|---|---|
committer | Madeline <46743919+MaddyUnderStars@users.noreply.github.com> | 2022-11-05 21:59:12 +1100 |
commit | ef2d739965a6887021b796140c0c7543b1587d63 (patch) | |
tree | cb6e7110854e97079ff24d3c83481dce7ad4d5ed | |
parent | Rewrite gateway message decoding (diff) | |
download | server-ef2d739965a6887021b796140c0c7543b1587d63.tar.xz |
Fix 404 handler
-rw-r--r-- | src/api/Server.ts | 7 |
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(); }, ); |