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

Allow passing custom dns_resolver to client.connect #120

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

daurnimator
Copy link
Owner

@daurnimator daurnimator commented Aug 22, 2018

By passing a custom dns_resolver object you can:

  • Use a custom nameserver
  • Specify host=>ip mapping manually using a cqueues dns.hosts object

It has been written as a base for connection pooling (#10) and Happy Eyeballs.

@coveralls
Copy link

coveralls commented Aug 22, 2018

Coverage Status

Coverage increased (+0.2%) to 86.9% when pulling 5a80e99 on client.dns_resolver into e7cb076 on master.

@daurnimator daurnimator force-pushed the client.dns_resolver branch 2 times, most recently from 86f0bc4 to 5a80e99 Compare August 23, 2018 16:23
@daurnimator
Copy link
Owner Author

The http.socks module should gain a similar ability.

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

Successfully merging this pull request may close these issues.

2 participants