Nest Router in V5 about nest-router HOT 7 CLOSED

Hi @thaoula 😀

Actually The RouterModule Should Work as Expected with the new Version of Nest v5

I also added an example to demonstrate that it works as usual
check it here

and about your issue I also added a LoggerMiddleware that works like a charm.
see it also LoggerMiddleware

from nest-router.

Hi @shekohex,

Really appreciate getting back to me and thanks for the example.

I was getting -

Then I changed forRoutes("*") to forRoutes("/") like you example and now it works.

Regards,
Tarek

from nest-router.

Glad it works.
Happy Coding 😄 !

from nest-router.

Actually, do you mind explaining the difference between forRoutes("*") and forRoutes("/"). Our application has a number of middleware registered like this.

It would be great to understand why it worked in V4 and not V5.

from nest-router.

Imo, i think because of this note

move from traditional express middleware model: each middleware is solely bounded to a particular path, regardless of the request method.

Nest Now not only supporting Express Middleware Model, Actually the design had to be changed to work well with new Adapters.
but anyway i think by just setting a global middleware in the root module. it should catch all routes, did you try this ? give it a shot.

from nest-router.

@shekohex your Note link is broken!

from nest-router.

@johnbendi
Thanks, Comment updated :)

from nest-router.