-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Add client ip property in transport common module #261
Conversation
Codecov Report
@@ Coverage Diff @@
## master #261 +/- ##
============================================
- Coverage 49.83% 46.22% -3.62%
- Complexity 915 924 +9
============================================
Files 158 181 +23
Lines 5259 5746 +487
Branches 757 830 +73
============================================
+ Hits 2621 2656 +35
- Misses 2306 2753 +447
- Partials 332 337 +5
Continue to review full report at Codecov.
|
...ransport-common/src/main/java/com/alibaba/csp/sentinel/transport/config/TransportConfig.java
Outdated
Show resolved
Hide resolved
And please add test cases for new added |
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
Thanks for contributing! |
spring.cloud.inetutils.preferred-networks=172. 没有这样的参数支持多网卡吗 |
Describe what this PR does / why we need it
增加客户度ip可配置。让docker环境部署时可以指定宿主机IP。
若配置了客户端ip则用配置的ip,否则用获取的本地host的ip
Does this pull request fix one issue?
Fixes #246