-
Notifications
You must be signed in to change notification settings - Fork 267
Open
Description
Hi,
I just found this. Niiice!
Looks like a good idea for a very clean, general purpose architecture. I took it for a drive test and I run into:
headers already send situation
The issue lies here:
public execute (req: express.Request, res: express.Response): void {
this.req = req;
this.res = res;
this.executeImpl();
}
If I convert
userRouter.post('/',
(req, res) => createUserController.execute(req, res)
)
to
userRouter.post('/', async (req, res) => {
const createUserController = new CreateUserController(/useCaseGoesHere/)
await createUserController.execute(req, res)
})
it work.
spravo, danielmbirochi, InfoLorenzo, SebaRiccardo and SayedDileri
Metadata
Metadata
Assignees
Labels
No labels