Hide Redis password in info logs when redis username is empty - Githubissues

ContentSquare / chproxy

Open-Source ClickHouse http proxy and load balancer

https://www.chproxy.org/

MIT License

1.24k stars 256 forks source link

Hide Redis password in info logs when redis username is empty #399

Closed smunukutla-mycarrier closed 5 months ago

smunukutla-mycarrier commented 5 months ago

Description

Hide Redis password in info logs when redis username is empty in chproxy config

Pull request type

Implements #398

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Checklist

[ ] Linter passes correctly
[ ] Add tests which fail without the change (if possible)
[ ] All tests passing
[ ] Extended the README / documentation, if necessary

Does this introduce a breaking change?

[ ] Yes
[x] No

Further comments

mga-chka commented 5 months ago

FYI, we should release a new version before then end of the month