darold/squidanalyzer

Default DNS timeout value is misconfigured (or explanation is wrong)

mysterfr opened this issue ยท 1 comments

Hello,

Great project, and more visually appealing than LightSquid ๐Ÿ‘

In the configuration file, DNSLookupTimeout is set to 0.0001 and comment states this means 100ms.
This is wrong.

100ms = 0.100s

When you set 0.0001 it actually means 100 ยตs (microsecond). (equals to 0.1ms)

This is way too short and makes DNS resolution systematically fail.
Setting must then be adjusted to

DNSLookupTimeout=0.1

(see table at the bottom of this page if the above isn't clear http://www.sengpielaudio.com/calculator-millisecond.htm)

Good catch. This part was totally confuse to simplify I have change the unit to use for DNSLookupTimeout now it must be set in milliseconds. Commit d42f108 fix that.

When updating be warned that this break the backward compatibility with old configuration files.