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

Curl resolver threaded fix lag #112

Closed
wants to merge 5 commits into from

Conversation

m6w6
Copy link
Contributor

@m6w6 m6w6 commented Aug 21, 2014

This is actually on top of PR #111

The resolver thread occasionally gets stuck in getaddrinfo() when the DNS or anything else is crappy or slow, so when a host is found in the DNS cache, leave the thread alone and let itself cleanup the mess.

m6w6 added 5 commits August 21, 2014 01:52
the threaded resolver is a stubborn sheep though,
it still sits there and waits 5 seconds to complete
100 google.com resolves
The docs reflected the first implementation; updated to reflect the end result.
don't know how well that plays on Windows, though
@m6w6
Copy link
Contributor Author

m6w6 commented Aug 21, 2014

@bagder
Copy link
Member

bagder commented Aug 31, 2014

Thanks, merged and pushed now (as a single squashed commit)

@bagder bagder closed this Aug 31, 2014
@m6w6 m6w6 deleted the curl_resolver_threaded_fix_lag branch August 31, 2014 12:32
@lock lock bot locked as resolved and limited conversation to collaborators Jan 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants