oss-taishan-ai commented on issue #552: URL: https://github.com/apache/rocketmq-connect/issues/552#issuecomment-4654322845
## Assessment by github-manager-bot **Type:** Bug **Priority:** Medium ### Summary User reports that when Redis requires authentication, the `getRedisUri()` method in `org/apache/rocketmq/connect/redis/common/Config.java` does not include the password in the URI, causing authentication failures. ### Analysis This is a valid bug. The Redis URI construction is missing the password component, which is required for authenticated Redis connections. The suggested fix is to add password authentication to the URI format: `redis://:password@host:port`. Additionally, the user notes that the `redis.replicator.version` may be outdated and only supports Redis 6. ### Suggestions 1. Update `getRedisUri()` to include password authentication in the URI format 2. Consider upgrading the Redis replicator version for better Redis version compatibility 3. Add unit tests for authenticated Redis connections ### Verdict **Valid bug** — Redis connector fails to handle authenticated connections. Requires code fix to include password in URI construction. --- *Automated assessment by github-manager-bot* -- 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]
