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

Initial request for NS set when the auth. nameservers refuse EDNS #1308

Open
bortzmeyer opened this issue Nov 13, 2024 · 2 comments · May be fixed by zonemaster/zonemaster-engine#1397
Open
Assignees
Labels
P-High Priority: Issue to be solved before other T-Bug Type: Bug in software or error in test case description
Milestone

Comments

@bortzmeyer
Copy link

https://zonemaster.fr/en/result/8d4ea300d6630f5b

mail.protection.outlook.com is badly broken. Both authoritative name servers reply FORMERR as soon as there is a EDNS option.

As a result, Zonemaster cannot start the tests. IMHO, Zomaster should retry its initial NS query without EDNS.

(There are other funny things with this server. They send the reply to a SOA query in Authority, not Answer.)

@tgreenx
Copy link
Contributor

tgreenx commented Nov 13, 2024

It looks like Zonemaster is sending EDNS queries by default in all cases, but it shouldn't. See https://github.com/zonemaster/zonemaster/blob/v2024.1/docs/public/specifications/tests/DNSQueryAndResponseDefaults.md.
The messages come from Basic02, which is specified to send a DNS Query.

Thanks for the report, I've already identified where the issue is, I'll provide a fix.

@tgreenx tgreenx self-assigned this Nov 13, 2024
@tgreenx tgreenx added T-Bug Type: Bug in software or error in test case description P-High Priority: Issue to be solved before other labels Nov 13, 2024
@tgreenx tgreenx added this to the v2024.2 milestone Nov 13, 2024
@matsduf
Copy link
Contributor

matsduf commented Nov 13, 2024

It looks like Zonemaster is sending EDNS queries by default in all cases, but it shouldn't.

I.e. not only for Basic02?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-High Priority: Issue to be solved before other T-Bug Type: Bug in software or error in test case description
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants