fix: [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client

Hi all. When I have the following case, routing controllers fails and try to execute both functions:

@JsonController()
export class UserController {

    @Get("/users/me")
    getMe() {
      const id = 1;
       return userRepository.findById(id);
    }

    @Get("/users/:id")
    getOne(@Param("id") id: number) {
       return userRepository.findById(id);
    }
}

As far as I read it should execute first occurency not both. Thanks!

[doghappy]

I have the same error. I use routing-controller in an existing express program. when I remove a middleware, it won’t throw again.

// catch 404 and forward to error handler
// app.use(function (req: Request, res: Response, next: NextFunction) {
//    res.status(404);
//    res.send({
//        code: 404,
//        message: "Not Found"
//    });
//});

[gruckion]

id is a number so you can use regex to be stricter on the matching.

@Get("/users/:id([0-9]+)")