-
Notifications
You must be signed in to change notification settings - Fork 543
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
fix ipv6 formatting for consul API #287
Conversation
+ add test
@knalx Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
@knalx Thank you for signing the Contributor License Agreement! |
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.
This change breaks two tests.
Results :
Tests in error:
getMetadataWorks(org.springframework.cloud.consul.discovery.ConsulDiscoveryClientCustomizedTests): java.net.UnknownHostException: testConsulDiscovery2Host: unknown error
testGetFilteredListOfServers(org.springframework.cloud.consul.discovery.HealthServiceServerListFilterTests): java.net.UnknownHostException: serviceaddrPASSING: unknown error
- fix exception checking - add test
Sorry, did not check it. Now it works. |
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.
We use tabs, not spaces
* pull287: Fixes ipv6 formatting for consul discovery.
Squash and merged with some polish via 0069798 |
When consul api return IPv6 address - spring cloud consul can't parse it correctly - using it as is.
Added IPv6 address format modification.