-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
ENOTFOUND and ECONNRESET errors #16
Comments
I agree. ETIMEDOUT is another one. I have done a few hacks to work around and get some stack in these instances but I would love to "fix" them once and for all. Once the contributing stuff ( #20 ) has been confirmed I might try to find some time to resolve. |
@defunctzombie I had a quick look and the usage of these codes is a bit all over the place. Might take a bit to actually come up with a good fix. In the meantime you can do something like this to give some additional debugging info when these errors occur. https://github.com/MauriceButler/request-callback-wrapper/blob/master/index.js Ill have another look on the weekend and try to work something out. |
Hello! I am pleased to see your valuable contribution to this project. Would you Questions:
Please provide the answers in an ordered list like this:
Note that I am just a bot with a limited human-reply parsing abilities, In case of success I will say: In case of validation problem I will say: Truly yours, Responsibilities
|
@defunctzombie since #35 has been merged, is this OK to close? |
😍 |
I will switch to anything that gives me better debug info when these two errors happen. Please just put the domain or IP or something in the message :)
The text was updated successfully, but these errors were encountered: