Middleware doesn't get called if more than one version is specified in `version` about nest

[kamilmysliwiec]

🤔 @micalevisk would you like to create a PR for this issue?

[1valdis]

Workaround:

export class AppModule {
  configure(consumer: MiddlewareConsumer) {
    ['1', '2'].forEach(version => consumer.apply(VersioningMiddleware).forRoutes({
      path: '*',
      version,
      method: RequestMethod.ALL,
    }));
  }
}

But I'd rather not 🥲

[micalevisk]

from what I tested, this is the behavior since v9.0.0 of @nestjs/core. But looks buggy to me

I notice that, in your example, the following line is resulting in the [ "/v1,2/*" ] instead of [ "/v1/*", "/v2/*" ] (I suppose?):

nest/packages/core/middleware/route-info-path-extractor.ts

Line 43 in 8cfa854

: [this.prefixPath + versionPath + addLeadingSlash(path)];

because of this version.toString() (['1', '2'].toString()):

nest/packages/core/middleware/route-info-path-extractor.ts

Line 85 in 8cfa854

return addLeadingSlash(versionPrefix + version.toString());

Might be related with that 🤔

[kamilmysliwiec]

Let's track this here #13193