-
Notifications
You must be signed in to change notification settings - Fork 8.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feature: support password decryption #2667
Conversation
server/src/main/java/io/seata/server/store/DruidDataSourceGenerator.java
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## develop #2667 +/- ##
=============================================
- Coverage 50.37% 50.21% -0.17%
- Complexity 3116 3123 +7
=============================================
Files 594 595 +1
Lines 19625 19699 +74
Branches 2438 2410 -28
=============================================
+ Hits 9887 9892 +5
- Misses 8744 8788 +44
- Partials 994 1019 +25
|
core/src/main/java/io/seata/core/store/db/AbstractDataSourceProvider.java
Outdated
Show resolved
Hide resolved
core/src/main/java/io/seata/core/constants/ConfigurationKeys.java
Outdated
Show resolved
Hide resolved
core/src/main/java/io/seata/core/store/db/AbstractDataSourceProvider.java
Outdated
Show resolved
Hide resolved
add publicKey to config.txt |
Codecov Report
@@ Coverage Diff @@
## develop #2667 +/- ##
=============================================
- Coverage 51.69% 51.66% -0.04%
- Complexity 3344 3354 +10
=============================================
Files 617 618 +1
Lines 20265 20340 +75
Branches 2539 2543 +4
=============================================
+ Hits 10477 10509 +32
- Misses 8737 8780 +43
Partials 1051 1051
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
server/src/main/java/io/seata/server/storage/redis/JedisPooledFactory.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for @caohdgege
你们好,我看所有关于加解密的都会被引导到这个地方,但是我没有在这个地方找到如何进行加解密的方式,或许这个文章中提到了一些关键词,但是我对这个方面的知识比较少,目前还是不知道怎么加解密,如果有人知道的话,请告知具体的方式,感谢 |
Hello, I see that everything about encryption and decryption leads to this place, but I didn't find how to encrypt and decrypt in this place. Maybe there are some keywords mentioned in this article, but I have little knowledge about this. I still don't know how to encrypt and decrypt. If anyone knows, please tell me how. Thanks. |
Ⅰ. Describe what this PR did
support druid encryption and decryption
Ⅱ. Does this pull request fix one issue?
fix #2665
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews