shreemaan-abhishek commented on PR #13670:
URL: https://github.com/apache/apisix/pull/13670#issuecomment-4912232164
Thanks @membphis, good catches. Fixed in the latest commit:
- Added \`encrypt_fields = {"redis_password", "sentinel_password"}\` to the
plugin schema so the Redis credentials are encrypted at rest, matching
\`limit-count\` / \`limit-req\`.
- Forwarded \`redis_username\` / \`redis_password\` in the
\`redis-sentinel\` branch of \`transform_limit_conf\` so a Redis master with
ACL/requirepass can be authenticated.
- Added a schema test for a sentinel config with Redis master auth, and a
test asserting \`redis_password\` is encrypted at rest.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
