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

invalid edns queries #1313

Open
jay2398743 opened this issue Dec 6, 2023 · 4 comments
Open

invalid edns queries #1313

jay2398743 opened this issue Dec 6, 2023 · 4 comments
Labels
T-Question Type: Incoming question

Comments

@jay2398743
Copy link

It looks like Zonemaster do not follow rfc during queries. When EDNS queries are not accepted causing FORMERR in response, a normal query should be used instead.

@matsduf matsduf added the T-Question Type: Incoming question label Dec 6, 2023
@matsduf
Copy link
Contributor

matsduf commented Dec 6, 2023

Thank you for your question. Zonemaster is a test tool. In Nameserver02 Zonemaster checks for EDNS support, and if FORMERR is returned, a message is returned to the client that there is no EDNS support for the specific name server. In that case Zonemaster does not requery since that is meaningless.

The Zonemaster queries have another purpose than most DNS queries from "normal" clients. Most test cases do the testing without EDNS, except where DNSSEC support in the query is needed.

Do you have something else in your mind? We might have missed something.

@jay2398743
Copy link
Author

jay2398743 commented Dec 6, 2023

Zonemaster reports critical error in system and basic (The domain must have at least one working name server) when using non-EDNS servers, and it stops after that doing no more tests.

Since this is the first test Zonemaster is doing, no tests are actually performed

@matsduf
Copy link
Contributor

matsduf commented Dec 7, 2023

@jay2398743, could you share what domain name (zone) you have tested so that I can investigate? Either here or in an email directly to me (mats.dufberg at iis.se)?

@matsduf matsduf transferred this issue from zonemaster/zonemaster Dec 9, 2023
@matsduf
Copy link
Contributor

matsduf commented Dec 9, 2023

See issue #1314

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Question Type: Incoming question
Projects
None yet
Development

No branches or pull requests

2 participants