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

general update + ability to set dns servers to use #6

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

Conversation

vodolaz095
Copy link

Thanks for good library.
But when i try to start it on nodejs project working in docker container, i got rate limit errors because of this
moby/libnetwork#2214
moby/libnetwork#2082

by default, nodejs uses default docker resolver (172.0.0.11) as described here
https://docs.docker.com/v17.09/engine/userguide/networking/configure-dns/

but this resolver has only 100 connections per second limit.
When i try to check domains by your library, it fails.

To be able to check with good speed, i use custom DNS servers.
and my PR allows us to add this servers via .setServers exported function.

also i fixed tests, updated code, integrated eslint, edtor config and so on.
Tests, of course, pass.

vodolaz095 added 3 commits May 1, 2019 17:41
2. eslint + linting code
3. npm test fixed (commented out lists not working)
4. add function `.setServers` to set custom DNS server being used for resolve.

.addServers is important, because in docker, this package hits rate limit
moby/libnetwork#2082
moby/libnetwork#2214
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.

1 participant