Skip to content

dns: default to verbatim=true in dns.lookup() #39987

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.

Sign up for GitHub

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

Closed
treysis wants to merge 18 commits into from
Closed

dns: default to verbatim=true in dns.lookup() #39987

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
05bbc0e
dns: default to verbatim=true in dns.lookup()
bnoordhuis Jan 29, 2020
acdfa26
dns: fix dns.md
treysis Sep 3, 2021
36c0fe2
dns: added suggestion from comments
treysis Sep 3, 2021
09e0021
dns: fix test suite, part 1
treysis Sep 4, 2021
b352e42
dns: fix test suite part 2
treysis Sep 4, 2021
7956d2d
dns: minor lint fix
treysis Sep 4, 2021
9ca2309
dns: fix lint error in test-net-pingpong.js
treysis Sep 4, 2021
32a4bca
dns: fix test suite, part 3
treysis Sep 4, 2021
5822f5e
dns: fix lint errors
treysis Sep 4, 2021
82af5ec
dns: fix test suite, part 4
treysis Sep 4, 2021
0cb3a69
dns: fix test-net-dns-lookup.js
treysis Sep 4, 2021
dba92ad
dns: test suite fixes for CI
treysis Sep 5, 2021
f5b3eaa
dns: fix linter errors
treysis Sep 5, 2021
6ac9106
dns: fix test/pummel/test-net-pingpong.js
treysis Sep 5, 2021
e385c66
dns: trying to fix test-net-remote-address-port.js
treysis Sep 5, 2021
431f7b2
dns: try to fix test suite for CI
treysis Sep 5, 2021
424f34f
dns: test suite fix for CI node-test-commit-aix
treysis Sep 5, 2021
559c3f3
dns: apply suggestions from code review
treysis Sep 5, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
dns: added suggestion from comments
  • Loading branch information
@treysis
treysis committed Sep 3, 2021
commit 36c0fe21ac3348a6383cce9fdbf31d7513390fbe
9 changes: 1 addition & 8 deletions lib/internal/dns/utils.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -196,14 +196,7 @@ function emitInvalidHostnameWarning(hostname) {
let dnsOrder = getOptionValue('--dns-result-order') || 'verbatim';

function getDefaultVerbatim() {
switch (dnsOrder) {
case 'verbatim':
return true;
case 'ipv4first':
return false;
default:
return true;
}
return dnsOrder !== 'ipv4first';
}

function setDefaultResultOrder(value) {
Expand Down