Not working for limit greater than 10_000 about nestjs-throttler-storage-redis

[Zippersk]

Is it possible to throw error when limit is less than scanCount? I think it would help...

[kkoomen]

If people run into the problem then I will increase it, maybe even allowing a configurable option for it.

[wips]

@kkoomen , thanks for the scanCount parameter, but it really took me awhile to find the issue. It especially nasty because locally you don't usually test with big limits as @Zippersk mentioned. I believe having an example in readme with full parameter set would be beneficial.

[kkoomen]

@wips This does work right? It has been implemented so I guess I can close this. I totally forgot to close this.

[wips]

@kkoomen , It works, thanks, but in a weird way. For example when I set LIMIT to 1k I need to set scanCount somewhere between 10k and 20k to make it reliably cut-off requests after limit is reached. It happens when I try to load the endpoint at 5-10req/s rate. I assume it has nothing to do with this project, looks like a weird behaviour of ioredis or Redis itself, I don't have much experience with it, so maybe it's expected.