-
Notifications
You must be signed in to change notification settings - Fork 120
feat(redis): support cloud Redis AuthN #205
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
Conversation
| auth = auth, | ||
| username = username, | ||
| password = password, | ||
| force_auth = force_auth, |
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.
ref: Kong/resty-redis-cluster#44
Only kong-resty-rediscluster will make this flag take effect.
0d17bc6 to
16ee522
Compare
| local ssl_verify = configuration and configuration.ssl_verify | ||
| local server_name = configuration and configuration.server_name | ||
|
|
||
| local force_auth = configuration and configuration.force_auth |
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.
| local ssl_verify = configuration and configuration.ssl_verify | ||
| local server_name = configuration and configuration.server_name | ||
|
|
||
| local force_auth = configuration and configuration.force_auth |
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.
Remove the read only restriction on object methods and add a flag to control whether auth is required after each connection.
16ee522 to
b00715f
Compare
force_redis_authto control whetherauthis required after each connection