ThrottlerStorage does not save the throttler object to map

hcminh commented 1 month ago

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior

I have followed this official document to install @nestjs/throttler. But it not working with the following config

app.module.ts

@Module({
  imports: [
    ThrottlerModule.forRoot([{
        ttl: 60000,
        limit: 3,
      }]),
    WebModule,
  ],
  controllers: [AppController],
  providers: [
    { provide: APP_GUARD, useClass: ThrottlerGuard },
    AppService,
  ],
})
export class AppModule {}

app.controller.ts

@Controller()
@Throttle({default: {limit: 3, ttl: 60000}})
export class AppController {
  constructor() {}

  @Get()
  hello(): string {
    return "hello"
  }
}

The response header I get overtime is

x-ratelimit-limit: 3
x-ratelimit-remaining: 2
x-ratelimit-reset: 60

Minimum reproduction code

n/a

Steps to reproduce

I have jumped to node_modules/nestjs/throttler to debug with console.log and found out that the throttlerStorage was not storing any upcoming requests. For any new request, function this.storage.get(key) always return undefined