feat: add redis and redis-cluster in limit-conn#10866
Conversation
|
Let's bring support for |
I'll split it into two pull requests. I encountered an issue during development related to the management of Redis keys. I noticed that APISIX generates a significant number of Redis keys. I'm considering consolidating these keys into a hash for better management. What are your thoughts on this? |
|
@shreemaan-abhishek @monkeyDluffy6017 I have a problem now. I need to write a test case to test the plugin, but what address should I fill in for the redis cluster? How to fill in the cluster with password and the cluster without password? How to fill in the address of the redis singleton with password and without password? |
|
@theweakgod you will find redis servers (singleton and cluster) defined as docker compose applications in |
Ok.I see, thank u. |
…requests in test cases
|
@shreemaan-abhishek @monkeyDluffy6017 I'm ready to review |
|
@shreemaan-abhishek I checked the doc and no more comments from me. |
8c142f1 to
f375fca
Compare
497c4ce to
03724a8
Compare
Description
Fixes #10501 (#10501)
Add options for redis and redis-cluster in the limit-conn plugin, and use redis as the counter for the leaky bucket algorithm.
Checklist