Add "username" setting support to Sentinel mode

[georgelupu]

Feature Request

Add "setSentinelUsername" method to Sentinel mode

Is your feature request related to a problem? Please describe

Latest Redis (6+) servers have ACL support which allows us to have in the database other users than the "default" one, but the current Lettuce client driver implementation does not allow to provide the "username" under RedisSentinelConfiguration class.
As result only the "default" db user can be used with the Lettuce driver in order to connect to Sentinel - which is a concern for DBOps folks who do not want to share "default" root/admin account with the application developers.

Describe the solution you'd like

Today, we can only provide setSentinelPassword config in RedisSentinelConfiguration class.
Please add "setSentinelUsername" method support as well. Thank you!

Describe alternatives you've considered

No alternatives are available at the moment except using the "default" redis db user for connecting to Sentinel.

[castlering97]

Hi @mp911de. I would like to work on this issue if it's okay.
(And I am a beginner in open source, so I would be really glad if you could give me any kind of guidance =])

[tishun]

Hi @mp911de. I would like to work on this issue if it's okay. (And I am a beginner in open source, so I would be really glad if you could give me any kind of guidance =])

Please submit a PR if you are still considering working on this. To do so the typical approach would be:

• fork the repository
• create a branchin your own fork for the fix (also known as topic branch)
• after you are ready locally you need to push to your forked branch
• when the push is done in the result you will get a link to file a pull request
• check out some of the existing pull requests for examples

[ggivo]

Referred configuration class RedisSentinelConfiguration is actually part of spring-data-redis.

It looks like support for providing a sentinel username in addition to the password is already added with commit
closing issue

Here is also more up-to-date link to RedisSentinelConfiguration (Spring Data Redis 3.3.5 API)