-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
Backward compatible support for IPv6 #1093
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1093 +/- ##
==========================================
+ Coverage 40.87% 41.36% +0.49%
==========================================
Files 58 58
Lines 4223 4233 +10
Branches 2313 2316 +3
==========================================
+ Hits 1726 1751 +25
+ Misses 1024 994 -30
- Partials 1473 1488 +15 ☔ View full report in Codecov by Sentry. |
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.
Maybe we can also set the "old" methods as deprecated
(with https://github.com/kiwix/libkiwix/blob/main/include/common.h#L7-L14)
Else, agree with this PR
It may be too soon. The nature of the enhancement is such that I feel fine with the old API staying around for some time. We can deprecate it later. |
- Restored kiwix::getNetworkInterfaces() API to the version before support for IPv6 was introduced - Renamed the new API method to kiwix::getNetworkInterfacesIPv4Or6()
The unit-tests only call the said functions and print their output which should then be examined by the maintainer.
1caa1fd
to
75bddbf
Compare
This PR brings backward compatibility to #1074