c-ares

Commit Graph

Author	SHA1	Message	Date
Brad House	74a64e486a	ares_timeout: missing thread-safety lock	5 months ago
Brad House	ef186fe2a4	remove unused SEND_QUAL_ARG2	5 months ago
Brad House	0d1008355a	remove unneeded warning disablement	5 months ago
Brad House	7d6de797d1	MSVC: Prevent C1041 error error C1041: cannot open program database '....'; if multiple CL.EXE write to the same .PDB file, please use /FS might be output in some conditions, add /FS compiler flag to prevent it. Fixes Issue: #796 Fix By: Brad House (@bradh352)	5 months ago
Brad House	cd1ff7b2f3	remove outdated comment	5 months ago
Brad House	350be9e1f1	mingw32 build fix	5 months ago
Brad House	325c75abd0	don't try to install removed header ares_rules.h	5 months ago
Brad House	e57a5b877b	remove duplicate typedef	5 months ago
Brad House	adc6659486	simplify header include order	5 months ago
Brad House	94b69fbd45	try to fix build	5 months ago
Brad House	9b6f197fec	warning in test	5 months ago
Brad House	381d38d925	forcibly kill bogus warning	5 months ago
Brad House	1b8cfdedc9	build fix	5 months ago
Brad House	bdd08d8dd4	fix warning	5 months ago
Brad House	4b0326e129	try to clean up some windows-specific warnings	5 months ago
Brad House	853244bc58	build fix	5 months ago
Brad House	54808a5190	fix comments	5 months ago
Brad House	7efb5019a2	fix build warnings	5 months ago
Brad House	44d030e40f	updates	5 months ago
Brad House	8293a05f63	cleanup more warnings due to new compiler flags	5 months ago
Brad House	b417a79b9b	fix	5 months ago
Brad House	253bdee3bc	enable more GCC/clang warnings, fix reported warnings	5 months ago
Brad House	bbcb1a2bdf	clang-format	5 months ago
Brad House	c2b91beaef	fix epoll flags usage	5 months ago
Brad House	a488525f08	Automatic query timeout adjustment based on server history (#794 ) With very little effort we should be able to determine fairly proper timeouts we can use based on prior query history. We track in order to be able to auto-scale when network conditions change (e.g. maybe there is a provider failover and timings change due to that). Apple appears to do this within their system resolver in MacOS. Obviously we should have a minimum, maximum, and initial value to make sure the algorithm doesn't somehow go off the rails. Values: - Minimum Timeout: 250ms (approximate RTT half-way around the globe) - Maximum Timeout: 5000ms (Recommended timeout in RFC 1123), can be reduced by ARES_OPT_MAXTIMEOUTMS, but otherwise the bound specified by the option caps the retry timeout. - Initial Timeout: User-specified via configuration or ARES_OPT_TIMEOUTMS - Average latency multiplier: 5x (a local DNS server returning a cached value will be quicker than if it needs to recurse so we need to account for this) - Minimum Count for Average: 3. This is the minimum number of queries we need to form an average for the bucket. Per-server buckets for tracking latency over time (these are ephemeral meaning they don't persist once a channel is destroyed). We record both the current timespan for the bucket and the immediate preceding timespan in case of roll-overs we can still maintain recent metrics for calculations: - 1 minute - 15 minutes - 1 hr - 1 day - since inception Each bucket contains: - timestamp (divided by interval) - minimum latency - maximum latency - total time - count NOTE: average latency is (total time / count), we will calculate this dynamically when needed Basic algorithm for calculating timeout to use would be: - Scan from most recent bucket to least recent - Check timestamp of bucket, if doesn't match current time, continue to next bucket - Check count of bucket, if its not at least the "Minimum Count for Average", check the previous bucket, otherwise continue to next bucket - If we reached the end with no bucket match, use "Initial Timeout" - If bucket is selected, take ("total time" / count) as Average latency, multiply by "Average Latency Multiplier", bound by "Minimum Timeout" and "Maximum Timeout" NOTE: The timeout calculated may not be the timeout used. If we are retrying the query on the same server another time, then it will use a larger value On each query reply where the response is legitimate (proper response or NXDOMAIN) and not something like a server error: - Cycle through each bucket in order - Check timestamp of bucket against current timestamp, if out of date overwrite previous entry with values, clear current values - Compare current minimum and maximum recorded latency against query time and adjust if necessary - Increment "count" by 1 and "total time" by the query time Other Notes: - This is always-on, the only user-configurable value is the initial timeout which will simply re-uses the current option. - Minimum and Maximum latencies for a bucket are currently unused but are there in case we find a need for them in the future. Fixes Issue: #736 Fix By: Brad House (@bradh352)	5 months ago
Brad House	0b50983539	CI: MSYS2: try to take advantage of package caching	5 months ago
Brad House	8d2d2fc542	missing files	5 months ago
Brad House	fbbc68797d	split windows sysconfig into its own file	5 months ago
Brad House	827a1d523c	ares_queryloop: output server list	5 months ago
Brad House	f8d1e63840	ares_queryloop: properly capture CTRL-C and cleanup	5 months ago
Brad House	7ea18a83b3	test: clean up some minor warnings	5 months ago
Brad House	8cded20f3a	update some build instructions	5 months ago
Brad House	f37cb2f4fa	fix link typo	5 months ago
Brad House	7b353da759	CI: MSYS: enable static analyzer	5 months ago
Brad House	d6534de510	disable static analyzer, doesn't work due to link issues	5 months ago
Brad House	d0ccb5efd8	CI: Msys: attempt to build with clang static analyzer	5 months ago
Brad House	732a3d0d73	CI: Msys: need different build dirs	5 months ago
Brad House	f9faa3f05c	try to work around windows ASAN issue by not using std::string::npos	5 months ago
Brad House	536f3bd42b	fix compiler flags	5 months ago
Brad House	0d301b5729	logic fix	5 months ago
Brad House	6d27b72b93	try to enable windows ASAN	5 months ago
Brad House	268092a390	MSVC: enable strict warnings (#792 ) MSVC has been building with /W3 which isn't considered a safe level for modern code. /W4 is recommended, but it too is lacking some recommended options, so we enable /W4 and also the recommended options. We do, however, have to disable a couple of options due to Windows headers not being fully compliant sometimes as well as some things we do in c-ares that it doesn't like, but aren't actually bad. Fix By: Brad House (@bradh352)	5 months ago
Brad House	d3642a6aeb	CI: Move some Appveyor builds to GitHub actions (#791 ) AppVeyor has gotten progressively slower over the last year or so. Move some Windows builds to GitHub actions which is considerably faster. Fix By: Brad House (@bradh352)	5 months ago
Brad House	1a5bd3b15c	CI: Add DJGPP (#789 ) Add automated building of DJGPP Fix By: Brad House (@bradh352)	5 months ago
Brad House	5899e96ec9	1.31.0 release prep (#788 )	5 months ago
Brad House	4248c642d2	Enable QueryCache by default (#786 ) The query cache should be enabled by default. This will help with determining proper timeouts for #736. It can still be disabled by setting the ttl to 0. There should be no negative consequences of this in real-world scenarios since DNS is based on the TTL concept and upstream servers will cache results and not recurse based on this information anyhow. DNS queries and responses are very small, this should have negligible impact on memory consumption. Fix By: Brad House (@bradh352)	5 months ago
Brad House	5804b30a87	CMAKE: Some systems might expect configure macros to be defined to 1 On some broken MacOS SDK versions, if HAVE_UNISTD_H is defined, it must be defined to 1, otherwise it will cause build errors. Autotools always defines to 1, but CMake doesn't. Fixes Issue: #787 Fix By: Brad House (@bradh352)	5 months ago
Brad House	266599a632	Windows: Reload config on manual DNS changes (#785 ) The `NotifyIpInterfaceChange()` method only notifies of automatic changes, such as via DHCP or interfaces going up or down. However, someone editing the DNS settings manually would not trigger a notification. Use `RegNotifyChangeKeyValue()` to monitor `HKLM\SYSTEM\CurrentControlSet\Services\Tcpip6\Parameters\Interfaces` and `HKLM\SYSTEM\CurrentControlSet\Services\Tcpip\Parameters\Interfaces` to pick up these changes. This new check also works when using the OpenWatcom compiler as the symbols are available, so they have been enabled (though `NotifyIpInterfaceChange()` method is disabled for OpenWatcom still due to the linker libraries missing this symbol). NOTE: it does appear that about 5 events get triggered for some reason, it doesn't seem to hurt anything, but it will actually cause it to re-read the configuration multiple times. Fixes Issue: #761 Fix By: Brad House (@bradh352)	6 months ago
Brad House	f05465e59b	tests: set ndots:1 as default, don't honor system config as it may skew results	6 months ago
Brad House	852a60a6ad	ndots:0 in system config not being honored due to bad sanity check	6 months ago

1 2 3 4 5 ...

653 Commits (74a64e486af72f08273446866f915db378773d07) All Branches Search

653 Commits (74a64e486af72f08273446866f915db378773d07)

All Branches