-
Notifications
You must be signed in to change notification settings - Fork 128
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: Issue/623 #706
fix: Issue/623 #706
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.
Thanks for your PR! 👍
LGTM 🚀
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #706 +/- ##
============================================
+ Coverage 88.33% 88.37% +0.04%
- Complexity 771 784 +13
============================================
Files 174 174
Lines 7088 7089 +1
Branches 389 390 +1
============================================
+ Hits 6261 6265 +4
+ Misses 700 698 -2
+ Partials 127 126 -1 ☔ View full report in Codecov by Sentry. |
Closes #623
Proposed Changes
Change the way in which
InfluxDBClientOptions
prepares a parsed URL, so that IPv6 addresses will be detected and preserved.Checklist
mvn test
completes successfully