Skip to content

Comments

Answer NXDOMAIN for _.xxx.yyy.top.domain#120

Merged
yarrick merged 1 commit intoyarrick:masterfrom
luizluca:nxdomain_for_underscore
Sep 4, 2025
Merged

Answer NXDOMAIN for _.xxx.yyy.top.domain#120
yarrick merged 1 commit intoyarrick:masterfrom
luizluca:nxdomain_for_underscore

Conversation

@luizluca
Copy link
Contributor

@luizluca luizluca commented Aug 3, 2025

When a DNS query name is too long, it is split into multiple domain components. However, recursive DNS servers that implement QNAME minimization may query each subdomain individually without revealing the full name. As part of this behavior, they often send a preliminary query for a hostname like "_" before attempting to resolve the full name. If this query is not handled correctly, it can lead to timeouts and failed connections. The most effective way to avoid this is to respond with an NXDOMAIN for such queries.

@luizluca
Copy link
Contributor Author

luizluca commented Aug 3, 2025

Closes #119

@luizluca luizluca force-pushed the nxdomain_for_underscore branch from ec813dd to b7aa67c Compare August 6, 2025 01:06
Copy link
Owner

@yarrick yarrick left a comment

Choose a reason for hiding this comment

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

See open comment

When a DNS query name is too long, it is split into multiple domain
components. However, recursive DNS servers that implement QNAME
minimization may query each subdomain individually without revealing the
full name.  As part of this behavior, they often send a preliminary
query for a hostname like "_" before attempting to resolve the full
name.  If this query is not handled correctly, it can lead to timeouts
and failed connections.  The most effective way to avoid this is to
respond with an NXDOMAIN for such queries.

Signed-off-by: Luiz Angelo Daros de Luca <luizluca@gmail.com>
@luizluca luizluca force-pushed the nxdomain_for_underscore branch from b7aa67c to 076ec9f Compare August 31, 2025 21:29
@luizluca luizluca requested a review from yarrick August 31, 2025 21:31
@yarrick yarrick merged commit 517d729 into yarrick:master Sep 4, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants