alibaba / alibaba/Sentinel

[BUG] 1.8.8 使用redis集群SSL数据源开启SSL失效

Open
#3,533 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
23.1k
Forks
8.1k
PR merge metrics
No merged PRs in 30d

Description

## Issue Description

Type: *bug report*

### Describe what happened

在RedisDateSource.java中,getRedisClusterClient() 方法中,生成的RedisURI并没有根据 connectionConfig的 sslEnable属性设置SSL,而是在循环体里面使用 connectionConfig.getRedisClusters的sslEnable设置,这导致SSL设置失效,因为最外层的RedisConnectionConfig只能设置sslEnable字段

Contributor guide

Open the contributing guide

Research direction

Open RedisDateSource.java and start with getRedisClusterClient(). Compare how the RedisURI is built with connectionConfig.sslEnable versus each entry from connectionConfig.getRedisClusters(). Done means SSL is honored when only the outer RedisConnectionConfig enables it.

Written by the indexing model from the issue text.

Assessment

Tech stack
java, redis
Domain
databases
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.