Skip to content
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

Win32 exclude legacy ipv6 subnets #144

Merged
merged 8 commits into from Sep 11, 2017
Merged

Win32 exclude legacy ipv6 subnets #144

merged 8 commits into from Sep 11, 2017

Conversation

@bradh352
Copy link
Member

@bradh352 bradh352 commented Sep 10, 2017

win32 ipv6: add infrastructure to exclude ipv6 subnets that are known to cause issues
Fixes issue #134

@bradh352
Copy link
Member Author

@bradh352 bradh352 commented Sep 10, 2017

@bagder @daviddrysdale any issues with me merging this?

@coveralls
Copy link

@coveralls coveralls commented Sep 10, 2017

Coverage Status

Coverage remained the same at 95.382% when pulling e325311 on bradh352:win32_exclude_legacy_ipv6 into cb68526 on c-ares:master.

@bradh352 bradh352 requested review from daviddrysdale and bagder Sep 11, 2017
@bagder
bagder approved these changes Sep 11, 2017
Copy link
Member

@bagder bagder left a comment

I'm fine with merging this, as I trust this is the right direction. I think you should squash/remove the debug commits before you merge...

@bradh352 bradh352 merged commit 9ef37fe into c-ares:master Sep 11, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 95.382%
Details
@bradh352 bradh352 deleted the bradh352:win32_exclude_legacy_ipv6 branch Sep 11, 2017
DronRathore added a commit to DronRathore/c-ares that referenced this pull request Mar 11, 2020
win32 ipv6: add infrastructure to exclude ipv6 subnets that are known to cause issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants