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

Zone01 crashes if two nameservers serve different SOA serials #1175

Closed
marc-vanderwal opened this issue Dec 23, 2022 · 4 comments · Fixed by #1178
Closed

Zone01 crashes if two nameservers serve different SOA serials #1175

marc-vanderwal opened this issue Dec 23, 2022 · 4 comments · Fixed by #1178
Assignees
Labels
P-High Priority: Issue to be solved before other S-PRforIssue Status: There is a PR that is meant to resolve the issue T-Bug Type: Bug in software or error in test case description
Milestone

Comments

@marc-vanderwal
Copy link
Contributor

Consider the following test zone, served by two nameservers but with different SOA serials:

$TTL 3600
$ORIGIN mname-not-master.zone01.xa.

@                       IN  SOA  (
                                   ns1.mname-not-master.zone01.xa.
                                   root.nic.test.
                                   2022122300 ; ns1 serves this serial, ns2 a slightly higher one
                                   86400
                                   14400
                                   3600000
                                   3600
                                 )

                            NS   ns1.mname-not-master.zone01.xa.
                            NS   ns2.mname-not-master.zone01.xa.

ns1                         A    127.0.1.1
ns2                         A    127.0.1.2

When running zonemaster-cli on such a broken zone, the test unfortunately crashes with a fatal error:

$ zonemaster-cli --test zone/zone01 --hints local.hints mname-not-master.zone01.xa
Seconds Level     Message
======= ========= =======
   0.04 CRITICAL  Fatal error in Zone: Undefined subroutine &Zonemaster::Engine::Test::Zone::max called at [redacted]/Zonemaster/Engine/Test/Zone.pm line 561.
@marc-vanderwal marc-vanderwal added the T-Bug Type: Bug in software or error in test case description label Dec 23, 2022
@marc-vanderwal marc-vanderwal added this to the v2022.2.1 milestone Dec 23, 2022
@matsduf
Copy link
Contributor

matsduf commented Dec 23, 2022

When running zonemaster-cli on such a broken zone, the test unfortunately crashes with a fatal error:

I would not even call it a broken zone. It is normal for two servers to serve different SOA serial just after an update of the zone. This a bad bug.

@tgreenx tgreenx added the P-High Priority: Issue to be solved before other label Dec 28, 2022
@tgreenx tgreenx self-assigned this Dec 28, 2022
@mdavids
Copy link

mdavids commented Jan 12, 2023

I encountered it as well:

https://zonemaster.net/en/result/64d899fe370d3c4f

@tgreenx tgreenx linked a pull request Jan 12, 2023 that will close this issue
@tgreenx
Copy link
Contributor

tgreenx commented Jan 12, 2023

I encountered it as well:

https://zonemaster.net/en/result/64d899fe370d3c4f

Hi @mdavids,

This will be fixed by #1178 in next fix release planned for 23/01/2023. But if you'd rather have a fix right now you can already look at the changes in the PR and implement them yourself.

@matsduf matsduf added the S-PRforIssue Status: There is a PR that is meant to resolve the issue label Jan 16, 2023
@tgreenx
Copy link
Contributor

tgreenx commented Jan 26, 2023

Solved by #1178

@tgreenx tgreenx closed this as completed Jan 26, 2023
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 S-PRforIssue Status: There is a PR that is meant to resolve the issue 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.

4 participants