-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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.
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
chore(response-rl): standarize redis configuration #12302
chore(response-rl): standarize redis configuration #12302
Conversation
40f8533
to
2d67545
Compare
89fc3b0
to
d519f2a
Compare
2d67545
to
b290eec
Compare
d519f2a
to
0b01009
Compare
6523fc1
to
c36ec13
Compare
955cd8b
to
3ef350f
Compare
f572e5d
to
fbfdef2
Compare
c36ec13
to
a877569
Compare
fbfdef2
to
a6b3bd6
Compare
a877569
to
1ef1dfc
Compare
1ef1dfc
to
e2938a7
Compare
a6b3bd6
to
3e79e98
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comment. If we resolve this, we should be good to go.
Response-RateLimiting right now has new config structure that reuses common redis connection configuration. The same as ACME and RateLimiting plugin. KAG-3388
3e79e98
to
6d5f720
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good now 👍🏼
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary
Response-RateLimiting right now has new config structure that reuses common redis connection configuration.
Many plugins differ in the way Redis is configured. The goal of this PR is to introduce a standard Redis config schema that could be shared across plugins.
Other PRs
This is the last in the chains of PRs that standardize Redis configuration.
Checklist
changelog/unreleased/kong
orskip-changelog
label added on PR if changelog is unnecessary. README.mdThere is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HEREIssue reference
KAG-3388