filebeat: fix Redis auth method to support username/password#44137
filebeat: fix Redis auth method to support username/password#44137rdner merged 2 commits intoelastic:mainfrom
Conversation
|
💚 CLA has been signed |
|
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
|
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
|
@lalit-satapathy it could be interesting to have someone from your team also reviewing this PR. |
rdner
left a comment
There was a problem hiding this comment.
This change needs a changelog entry in https://github.com/elastic/beats/blob/main/CHANGELOG.next.asciidoc
0a2ab61 to
1d9fc29
Compare
|
@rdner CHANGELOG.next.asciidoc added, thanks! |
rdner
left a comment
There was a problem hiding this comment.
Looks good to me in the current state.
As was pointed out, we use the same approach in the Metricbeat module here
beats/metricbeat/module/redis/redis.go
Line 218 in 6e3634f
|
/test |
Proposed commit message
filebeat: fix Redis auth method to support username/password
Reference: Redis AUTH Documentation
Checklist
CHANGELOG.next.asciidocorCHANGELOG-developer.next.asciidoc.Disruptive User Impact
Author's Checklist
How to test this PR locally
Run
redis_integration_test.goto verify changes.Related issues
Use cases
Screenshots
Logs