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

Adding singleton option to DNS for getaddrinfo bug. #1583

Closed
wants to merge 1 commit into from
Closed

Adding singleton option to DNS for getaddrinfo bug. #1583

wants to merge 1 commit into from

Conversation

fusiondog
Copy link
Contributor

A simple fix for a problem in some versions of glibc

@fusiondog
Copy link
Contributor Author

This is a relatively small change. I think it failed the check because of other issues you were having with Travis timeout recently.

@slackpad
Copy link
Contributor

slackpad commented Feb 9, 2016

@fusiondog I think so as well. Do you mind adding a note about this to the docs as well? I'm happy to do that, too - just let me know.

@sean-
Copy link
Contributor

sean- commented Feb 9, 2016

@fusiondog Thank you for your PR. I'm going to take PR and update it so that an administrator can specify an int instead of a bool to limit the number of addresses returned.

@fusiondog
Copy link
Contributor Author

@sean- Would the int be able to exceed the default 3 results? The doc update should probably still make a point of the getaddrinfo bug and that using the setting at value 1 is needed to resolve.

sean- added a commit that referenced this pull request Feb 12, 2016
Based on work done by @fusiondog in #1583, extend the concept to use an integer instead of a boolean.

Fixes: #1583 && #1481
sean- added a commit that referenced this pull request Mar 30, 2016
Based on work done by @fusiondog in #1583, extend the concept to use an integer instead of a boolean.

Fixes: #1583 && #1481
@sean- sean- closed this in #1712 Mar 30, 2016
@sean-
Copy link
Contributor

sean- commented Mar 30, 2016

This has been committed in #1712, thank you for your PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants