addLeadingSlash shoudl not add leading slash for optional parameter

[dcharbonnier]

Is there an existing issue for this?

• I have searched the existing issues

Current behavior

Due to recent route change addLeadingSlash fail to detect leading slash
The path @Get('{/:optional}/test.js') is now mapped to /{/:optional}/test.js this fail to match the path
/test.js the leading slash is present but optional.
I can make a PR for this issue but this looks dependant of the adapter so it may break fastify.
pillarjs/path-to-regexp#344 (reply in thread)

Minimum reproduction code

https://codesandbox.io/p/devbox/sharp-gould-8xyrf9?file=%2Fsrc%2Fapp.controller.ts

Steps to reproduce

No response

Expected behavior

/fail2.js ans /test/fail2.js should both work

Package

• I don't know. Or some 3rd-party 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 (see below)

Other package

No response

NestJS version

No response

Packages versions

│ ├── @nestjs/[email protected]
│ ├── @nestjs/[email protected]
│ ├── @nestjs/[email protected]
│ ├── @nestjs/[email protected]
│ ├── @nestjs/[email protected]
│ ├── @nestjs/[email protected]
│ ├── @nestjs/[email protected]

Node.js version

No response

In which operating systems have you tested?

• macOS
• Windows
• Linux

Other

No response