-
-
Notifications
You must be signed in to change notification settings - Fork 7.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WARN [LegacyRouteConverter] Unsupported route path: "/v1/(.*)". - With Fastify adapter #14568
Comments
What if you change it to |
Hey @kamilmysliwiec thank you for the answer. Same warning
|
Yeah, it looks like something is "wrong" in my project, updating the minimum reproduction, the warning goes away |
Found the issue, I added the packages nestjs-pino + pino-http, for logging I added it, like so:
Likely it is still not compatible with nestjs 11. @kamilmysliwiec feel free to close the issue Thank you! |
Isn't it related to this? iamolegga/nestjs-pino#1849 |
Is there an existing issue for this?
Current behavior
likely tied to #14513
I am seeing the same warning, however I am using Fastify, not Express
Here is my setup:
app.setGlobalPrefix('v1');
Changing the path: '(.*)' to path: '{*path}' did not work
Thanks!
Minimum reproduction code
https://github.com/rafakwolf/nest-test
Steps to reproduce
yarn install
yarn start:dev
Expected behavior
I expect not see the warning message
Package
@nestjs/common
@nestjs/core
@nestjs/microservices
@nestjs/platform-express
@nestjs/platform-fastify
@nestjs/platform-socket.io
@nestjs/platform-ws
@nestjs/testing
@nestjs/websockets
Other package
No response
NestJS version
No response
Packages versions
Node.js version
v20.17.0
In which operating systems have you tested?
Other
No response
The text was updated successfully, but these errors were encountered: