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

enforce a maximum DNS cache size independent of timeout value #11084

wants to merge 2 commits into from


Copy link

@bagder bagder commented May 8, 2023

To prevent it from growing uncontrollably.

... because it set and held as an 'int' elsewhere and can never be
@bagder bagder added the name lookup DNS and related tech label May 8, 2023
To reduce the damage an application can cause if using -1 or other
ridiculous timeout values and letting the cache live long times.

The maximum number of entries in the DNS cache is now totally
arbitrarily and hard-coded set to 29999.
@bagder bagder changed the title hostip: store dns timeout as 'int' enforce a maximum DNS cache size independent of timeout value May 8, 2023
@bagder bagder closed this in 9ed7d56 May 8, 2023
@bagder bagder deleted the bagder/dns-timeout-int branch May 8, 2023 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
name lookup DNS and related tech
None yet

Successfully merging this pull request may close these issues.

None yet

1 participant