Skip to content
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.

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

DNS Resolver should return NXDOMAIN #1873

Closed
euh2 opened this issue Apr 20, 2024 · 0 comments · Fixed by #1877
Closed

DNS Resolver should return NXDOMAIN #1873

euh2 opened this issue Apr 20, 2024 · 0 comments · Fixed by #1877
Labels
bug Something isn't working

Comments

@euh2
Copy link

euh2 commented Apr 20, 2024

The problem

The Netbird client adds its domain as a search domain in /etc/resolv.conf. That's all good and fine. But many applications go through all search domains when trying to resolve an address. That means that e.g. a query for ww.google.com will hit the local Netbird resolver with www.google.com.netbird.cloud. Netbird then returns an empty A record with a NOERROR flag.
The application then stops lookup with "address not found" or similar.

> dig www.google.com.netbird.cloud.

; <<>> DiG 9.18.24 <<>> www.google.com.netbird.cloud.
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 7884
;; flags: qr rd ra; QUERY: 1, ANSWER: 0, AUTHORITY: 0, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 65494
;; QUESTION SECTION:
;www.google.com.netbird.cloud.   IN      A

;; Query time: 1 msec
;; SERVER: 127.0.0.53#53(127.0.0.53) (UDP)
;; WHEN: Sat Apr 20 18:34:30 CEST 2024
;; MSG SIZE  rcvd: 56

(I replaced my custom domain with "netbird.cloud")

If the local Netbird resolver would return NXDOMAIN, the application would continue its lookup. Which it absolutely should do!

To Reproduce

See above.

Expected behavior

The local Netbird resolver should return NXDOMAIN when no valid record is found.

Are you using NetBird Cloud?

Self-hosted

NetBird version

0.27.3

NetBird status -d output:

Not applicable

Screenshots

Not applicable

Additional context

Anything I forgot to mention?

@pascal-fischer pascal-fischer added bug Something isn't working and removed triage-needed labels Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants