dns: set timeout to 1000ms when timeout <= 0

[theanarkh]

c-ares set timeout to DEFAULT_TIMEOUT(2000ms) When timeout is <= 0, so we should schedule timer every 1000ms.

if (optmask & ARES_OPT_TIMEOUTMS) {
    /* Apparently some integrations were passing -1 to tell c-ares to use
     * the default instead of just omitting the optmask */
    if (options->timeout <= 0) {
      optmask &= ~(ARES_OPT_TIMEOUTMS);
    } else {
      channel->timeout = (unsigned int)options->timeout;
    }
}

if (channel->timeout == 0) {
  channel->timeout = DEFAULT_TIMEOUT;
}

cc @jasnell

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

[nodejs-github-bot]

Review requested:

• @nodejs/net

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.19%. Comparing base (cd68e35) to head (1a25e88).
Report is 228 commits behind head on main.

Files with missing lines	Patch %	Lines
src/cares_wrap.cc	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #58441      +/-   ##
==========================================
- Coverage   90.19%   90.19%   -0.01%     
==========================================
  Files         635      635              
  Lines      187224   187223       -1     
  Branches    36765    36763       -2     
==========================================
- Hits       168876   168867       -9     
- Misses      11131    11133       +2     
- Partials     7217     7223       +6     

Files with missing lines	Coverage Δ
src/cares_wrap.cc	54.35% <0.00%> (-0.04%)	⬇️

... and 28 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/67066/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/67074/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/67084/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/67147/

[nodejs-github-bot]

Landed in f08e0a0