DNS fixes #15864

heimskr · 2024-12-19T02:02:57Z

What does this PR do?

Adds some DNS tests and the changes that caused them to start passing.

Documentation or TypeScript types
Code changes

How did you verify your code works?

Added tests from Node.

…lers

robobun · 2024-12-19T02:03:07Z

^{Updated 12:08 AM PT - Dec 27th, 2024}

❌ @heimskr, your commit fee3983 has 7 failures in #8561:

test/js/third_party/grpc-js/test-server-errors.test.ts - SIGILL on 🐧 20.04 x64-baseline
test/js/third_party/grpc-js/test-server-errors.test.ts - segmentation fault on 🐧 24.04 x64-baseline
test/js/third_party/grpc-js/test-server-errors.test.ts - SIGILL on 🐧 12 x64-baseline
test/js/third_party/grpc-js/test-server-errors.test.ts - SIGILL on 🐧 20.04 x64
test/js/web/fetch/fetch.test.ts - 1 failing on 🍎 13 aarch64
test/js/third_party/grpc-js/test-server.test.ts - SIGILL on 🍎 14 x64
test/js/third_party/grpc-js/test-server.test.ts - SIGILL on 🐧 12 x64-baseline
test/js/third_party/grpc-js/test-server.test.ts - segmentation fault on 🐧 3.20 x64-baseline
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🍎 13 aarch64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🍎 14 aarch64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 22.04 aarch64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 20.04 aarch64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 12 x64-baseline
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 24.04 x64-baseline
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 22.04 x64-baseline
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 22.04 x64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 20.04 x64-baseline
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 20.04 x64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 12 x64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 24.04 x64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 3.20 aarch64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 3.20 x64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 12 aarch64
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 3.20 x64-baseline
test/js/node/test/parallel/test-dns-channel-timeout.js - code 1 on 🐧 24.04 aarch64
test/js/node/dns/node-dns.test.js - 15 failing on 🍎 14 aarch64
test/js/node/dns/node-dns.test.js - 15 failing on 🍎 13 aarch64
test/js/node/dns/node-dns.test.js - 15 failing on 🍎 14 x64
test/js/node/dns/node-dns.test.js - 15 failing on 🍎 13 x64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 22.04 aarch64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 20.04 aarch64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 12 x64-baseline
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 24.04 x64-baseline
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 22.04 x64-baseline
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 20.04 x64-baseline
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 22.04 x64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 20.04 x64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 24.04 x64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 12 x64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 3.20 aarch64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 3.20 x64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 12 aarch64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 24.04 aarch64
test/js/node/dns/node-dns.test.js - 15 failing on 🐧 3.20 x64-baseline
test/js/node/test/parallel/test-dns.js - code 1 on 🍎 13 aarch64
test/js/node/test/parallel/test-dns.js - code 1 on 🍎 14 aarch64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 12 aarch64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 20.04 aarch64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 24.04 aarch64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 22.04 aarch64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 20.04 x64-baseline
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 24.04 x64-baseline
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 12 x64-baseline
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 22.04 x64-baseline
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 22.04 x64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 12 x64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 24.04 x64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 3.20 aarch64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 20.04 x64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 3.20 x64
test/js/node/test/parallel/test-dns.js - code 1 on 🐧 3.20 x64-baseline
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🍎 13 aarch64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🍎 14 aarch64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🍎 13 x64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🍎 14 x64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 12 aarch64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 22.04 aarch64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 24.04 aarch64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 22.04 x64-baseline
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 12 x64-baseline
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 20.04 aarch64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 24.04 x64-baseline
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 20.04 x64-baseline
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 24.04 x64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 12 x64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 20.04 x64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 22.04 x64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 3.20 aarch64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 3.20 x64
test/js/bun/dns/resolve-dns.test.ts - 18 failing on 🐧 3.20 x64-baseline

🧪 try this PR locally:

bunx bun-pr 15864

TODO: ref the DNSResolvers when creating promises

heimskr added 25 commits December 12, 2024 17:43

Add dns.setServers

98d2a9e

Special case for falsey hostnames in dns.lookup

8052a0e

Add an argument to ERR_INVALID_IP_ADDRESS

5587ac0

Expose AI_ALL, AI_ADDRCONFIG and AI_V4MAPPED from netdb.h under dns

d3b13e3

Better DNS lookup

69a5b10

Better DNS error objects

9019d7a

Initial work on dns.Resolver isolation

9a15f30

Isolated resolution

5ab64c0

Implement resolveAny via horrible callback hell

a5805bc

CAA record support

f5cc373

Glorious promises

c729dbb

Remove a comment that's not quite true anymore

8bafda4

Simplify some Zig

130aa1b

Some dns.ts fixes

a62e2dc

Simplify some Zig again

56b5141

Isolate promisified Resolver objects

b2a76cc

No more NAPTR in resolveAny

c7c22c4

There isn't actually a _socket parameter to dgram sockets' error hand…

e8e19c7

…lers

Add ENOSERVER

f4e30f9

Don't do regular lookup for A/AAAA

01e9cbf

Merge branch 'main' of github.com:oven-sh/bun into kai/dns-test-fixes

e2d9f59

Fix A record resolution

35a0c6d

Add test-dns.js

ec09c1f

Add test-dns-get-server.js

e2160db

Customizable timeout and tries options for DNS resolvers

3aa930e

heimskr requested review from a team and zackradisic and removed request for a team December 19, 2024 02:06

Fix Windows compilation probably

89499ad

heimskr added 30 commits December 19, 2024 15:59

Don't overwrite "name" properties of warning objects

eff93bd

Honor type and code parameters of process.emitWarning

a8d1082

Properly emit invalid hostname warnings

c9438ff

Add test-dns-lookupService.js

0e6bdaf

Implement Resolver.cancel()

a415b60

Add test-dns-channel-cancel-promise.js

9047047

Per-resolver reversing

922050c

Initial work on DNS timeout support

166a55a

TODO: ref the DNSResolvers when creating promises

Proper refcounting for DNSResolvers

8bf4542

Fix test-dns.js

606b05f

Better ref()ing

ff7f4bb

Add test-dns-channel-timeout.js

8672289

Add support for default DNS lookup result order

cbc971c

Obey DNS result order

ba1322f

Implement --dns-result-order

2e17364

Update test-dns-default-order-ipv{4,6}.js

2792588

Add test-dns-lookup-promises-options-deprecated.js

f17eb8d

Fix test-dns-lookup.js

5429471

Add test-dns-lookupService-promises.js

c1fc9cf

Add test-dns-multi-channel.js

48dfa29

Add error codes to dns.promises

a7b4f64

Add test-dns-resolve-promises.js

34eceb4

Make rrtype default to A if unspecified

25461e5

Use SystemError in c_ares.zig

232fd73

Reimplement resolveAny with an actual ANY query

1a82209

Validate hostname type

a6862b1

Add test-dns-set-default-order.js

9733f0e

Implement dns.Resolver.setLocalAddress

cae9d2d

Fix some memory leaks

9448f18

Don't allow setting DNS servers while any queries are active

fee3983

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

DNS fixes #15864

DNS fixes #15864

heimskr commented Dec 19, 2024

robobun commented Dec 19, 2024 •

edited

Loading

DNS fixes #15864

Are you sure you want to change the base?

DNS fixes #15864

Conversation

heimskr commented Dec 19, 2024

What does this PR do?

How did you verify your code works?

robobun commented Dec 19, 2024 • edited Loading

robobun commented Dec 19, 2024 •

edited

Loading