-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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 passing contextPath
parameter to Nacos client
#5111
Conversation
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.
Codecov Report
@@ Coverage Diff @@
## develop #5111 +/- ##
==========================================
Coverage 49.35% 49.35%
- Complexity 4145 4147 +2
==========================================
Files 737 737
Lines 26190 26198 +8
Branches 3232 3232
==========================================
+ Hits 12925 12931 +6
- Misses 11881 11885 +4
+ Partials 1384 1382 -2
|
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
config/seata-config-nacos/src/main/java/io/seata/config/nacos/NacosConfiguration.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
contextPath
parameter to Nacos client
…o 1114_for_5073 * '1114_for_5073' of https://github.com/zw201913/seata: bugfix: hikari datasource auto proxy fail (apache#5134) bugfix: rollback active xa connection fail (apache#5131) optimize: support oracle on delete tccfence logs (apache#5124) feature: support passing `contextPath` parameter to Nacos client (apache#5111) bugfix:NPE caused when there is no @GlobalTransactional annotation on the RM side (apache#5109) bugfix: Druid disable oracle implicit cache (apache#5098) bugfix: fix access key loss after server restart (apache#5097) optimize: remove druid dependency in ConnectionProxy (apache#5104) bugfix:fix ClassNotFoundException during the ZK unit test (apache#5101) bugfix: fix when seata and jpa are used together, their AutoConfiguration order is incorrect (apache#5092) optimize: lock priority attempts to insert (apache#4681) bugfix: update join condition placeholder param error (apache#5052)
Ⅰ. Describe what this PR did
Support passing contextPath parameter to Nacos client
Ⅱ. Does this pull request fix one issue?
fixes #5110
fixes #5012
Ⅲ. Why don't you add test cases (unit test/integration test)?
I provided some screenshots of the test
Ⅳ. Describe how to verify it
Use nacos as discovery center and config center,and set the value of contextPath.
Ⅴ. Special notes for reviews