VermaShivansh / coraza-ratelimit-plugin

5 stars 1 forks source link

Distributed systems #10

Closed VermaShivansh closed 1 year ago

jcchavezs commented 1 year ago

Two things:

VermaShivansh commented 1 year ago

Two things:

  • A good test here would be to run concurrent requests and then count the values in redis directly.
  • We really need to run tests with ab to make sure connections are handled correctly.

Actually, the magefile has a test script for testing distributed rate-limit just like you asked. It starts 3 instances and hits 1 request/second on it from the client and logs the syncedState from redis after each sync. I tested it and it was working well. I would request you to run that test, modify the secrule and test once if it looks good according to you.