Skip to content
This repository has been archived by the owner on Nov 17, 2018. It is now read-only.

Handling DNS TTLs? #44

Closed
martincostello opened this issue Dec 15, 2017 · 4 comments
Closed

Handling DNS TTLs? #44

martincostello opened this issue Dec 15, 2017 · 4 comments
Labels

Comments

@martincostello
Copy link
Contributor

Having just read through this (long) issue relating to HttpClient and DNS TTLs, there are comments made about pooling of clients/handlers so that over time cached IP addresses from DNS entries are evicted. If if understood correctly, pooling is something that's been at least mentioned in this repo, but I'm guessing probably from the socket-exhaustion point-of-view.

Is period flushing of shared instances from a DNS point-of-view something that's being explicitly considered as a use case for things that HTTP client factory would be able to do to help consumers fall into the pit of success for? If not, is it worth incorporating into the design?

@mkArtakMSFT
Copy link
Member

@glennc, @rynowak any thoughts on this?

@glennc
Copy link
Member

glennc commented Jan 9, 2018

Hey @martincostello, our design for this today takes into account the DNS caching that the native handler does. We will recycle handlers periodically to make sure that the DNS entries aren't cached forever.

We are also working with some corefx folks on a managed handler that will hopefully resolve this issue for us, meaning that we don't have to worry about it when using that handler. I don't know that we will completely fix these issues, but we are aware of them and are taking it into account in our designs.

@martincostello
Copy link
Contributor Author

@glennc Thanks for the information Glenn 👍

@glennc
Copy link
Member

glennc commented Jan 9, 2018

No problems.

@glennc glennc closed this as completed Jan 9, 2018
@glennc glennc added the question label Jan 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants