-
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
optimize: override eureka getHostName() return ipAddress #2192
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2192 +/- ##
==========================================
Coverage 54.37% 54.37%
Complexity 2469 2469
==========================================
Files 447 447
Lines 14749 14749
Branches 1734 1734
==========================================
Hits 8020 8020
Misses 5968 5968
Partials 761 761 |
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
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
Ⅰ. Describe what this PR did
在 通过eureka作为注册中心过程中发现注册到 eureka 中,健康检查和状态等地址中获取的均为服务器的hostname而非对应ip。通过 CustomEurekaInstanceConfig 中重写getHostName()方法修改为获取ip来解决该问题。
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews