change dictionary

[abbassep3200]

I think in projects with more than a few thousand simultaneous users we need that change. What do you think?

[cristipufu]

I fail to see the need for rulesDict to be thread safe. Do you mind sharing more details? Is this actually an issue? How can I reproduce it, etc?

[abbassep3200]

We have a scenario in which thirty thousand simultaneous requests are sent to the server per second. Our concern now is that in such a volume of requests this library will reduce server performance. I want to ask you whether this library has been tested under heavy loads?
Thanks

[cristipufu]

Each request should have its own dictionary instance, so this shouldn't be an issue.

Regarding the performance however, if all those requests are hitting the same rate limit, there's a lock in place to make sure that the limit is not overcome, so yes, that will reduce server performance.

If you're looking for something to work under heavy load, with multiple app instance nodes, you should implement something like this: #83 (comment) (using Redis INCR)