-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
Recursive addresses for #51
Conversation
This is a potential fix for #31 |
@ioquatix I can take a look at this, but probably not until tomorrow. |
@petergoldstein Awesome, just if you have time. The more I think about this PR, the more I am happy with it. The only think I'm not completely happy with is the Resolv::DNS::Name monkey patching.. initially it was useful but now not so much, it could probably be removed ( |
Fixes #31 |
@petergoldstein what do you think of this approach? Most existing functionality should be the same.