实现 #3000 Issue的功能：Redis 数据源支持 SSL #3045
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
sczyh30
added
to-review
|
cc @BooAndBoCai @alibaba/sentinel-committer for review |
|
@z521598 Could you please resolve the test failure? |
Of Course. |
Has been fixed. |
|
Thanks for contributing! |
LearningGp
pushed a commit
that referenced
this pull request
Dec 28, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe what this PR does / why we need it
实现 #3000 Issue的功能：Redis 数据源支持 SSL
Does this pull request fix one issue?
只有实现 #3000 Issue的功能：Redis 数据源支持 SSL
Describe how you did it
1）将lettuce版本从5.0.1升级至6.2.1，为了支持Redis SSL双向认证时的PEM格式的客户端证书与客户端私钥
2）在RedisConnectionConfig加入必要的参数，对接lettuce。
2.1）支持JSK格式，同时也支持PEM格式。
2.2）根据使用者的配置决定是采用 单向认证 还是 双向认证。
Describe how to verify it
Unit Test
Special notes for reviews