Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change(limit-count): ensure redis cluster name is set correctly #3910

Merged
merged 15 commits into from
Apr 7, 2021
Merged

change(limit-count): ensure redis cluster name is set correctly #3910

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
85407f7
fix the problem of multiple different redis-cluster data sources, str…
Mar 25, 2021
9cc3be1
set redis cluster name by etcd config
Mar 29, 2021
f7a5436
set redis cluster name by etcd config
Mar 30, 2021
3c1035f
redis_cluster_name is required parameter
Mar 30, 2021
bab152b
fix bad style
han6565 Mar 30, 2021
c554a1b
fix English grammar issues
han6565 Mar 30, 2021
499d6fa
test for redis_cluster_name
Mar 30, 2021
257d1de
update the example
Mar 30, 2021
b5ce549
Merge remote-tracking branch 'upstream/master'
Mar 31, 2021
0aa6ed6
Merge remote-tracking branch 'upstream/master'
Apr 1, 2021
0157968
update the doc
Apr 2, 2021
338d2c4
update the doc
Apr 2, 2021
c77b409
Merge remote-tracking branch 'upstream/master'
Apr 2, 2021
33b65a0
Merge remote-tracking branch 'upstream/master'
Apr 6, 2021
eb88001
update the doc
Apr 6, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions apisix/plugins/limit-count.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,9 @@ local schema = {
redis_timeout = {
type = "integer", minimum = 1, default = 1000,
},
redis_cluster_name = {
han6565 marked this conversation as resolved.
Show resolved Hide resolved
type = "string", default = "apisix-redis-cluster",
},
},
required = {"redis_cluster_nodes"},
han6565 marked this conversation as resolved.
Show resolved Hide resolved
}
Expand Down
3 changes: 2 additions & 1 deletion apisix/plugins/limit-count/limit-count-redis-cluster.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,8 @@ local mt = {

local function new_redis_cluster(conf)
local config = {
name = "apisix-redis-cluster",
-- can set different name for different redis cluster
han6565 marked this conversation as resolved.
Show resolved Hide resolved
name = conf.redis_cluster_name,
serv_list = {},
read_timeout = conf.redis_timeout,
auth = conf.redis_password,
Expand Down