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 | 4469acd3906d64cbc3d3e0f08a13475daef315ec (patch) | |
tree | 3474b31bb535655728fad281c85e5ebc076698cf | |
parent | Rewrite gateway message decoding (diff) | |
download | server-4469acd3906d64cbc3d3e0f08a13475daef315ec.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(); }, ); |