Skip to content

dns: validate default c-ares servers on first query#62411

Open
mcollina wants to merge 1 commit into
nodejs:mainfrom
mcollina:fix-dns-resolvesrv-first-query
Open

dns: validate default c-ares servers on first query#62411
mcollina wants to merge 1 commit into
nodejs:mainfrom
mcollina:fix-dns-resolvesrv-first-query

Conversation

@mcollina
Copy link
Copy Markdown
Member

Fixes #62326

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. cares Issues and PRs related to the c-ares dependency or the cares_wrap binding. needs-ci PRs that need a full CI run. labels Mar 23, 2026
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is (expectedly) failing outside of Windows

@niridya
Copy link
Copy Markdown

niridya commented May 11, 2026

I can still reproduce the issue on the current public Node.js v26.x release on Windows (dns.resolveSrv()ECONNREFUSED).

Also updated #62326 to mention that Node.js 26 is affected as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. cares Issues and PRs related to the c-ares dependency or the cares_wrap binding. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dns.resolveSrv returns ECONNREFUSED on Windows since v24.13.0 (still broken in v25.x and v26.x)

6 participants