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

When testing a zone with 2 NS that point to the same IP... #140

Closed
einar opened this Issue Nov 5, 2015 · 6 comments

Comments

Projects
None yet
3 participants
@einar
Contributor

einar commented Nov 5, 2015

...you correctly get a NOTICE due to DELEGATION02 that says "SAME_IP_ADDRESS" (this is cool and actually more detailed than DNSCheck), the problem is that DELEGATION01 has already passed this domain as this test only checks the names then continues.

So what Zonemaster misses is that this domain only has 1 NS, this is not the case with DNSCheck so something's missing.

You can try on "rayceem.com" for example.

@matsduf

This comment has been minimized.

Show comment
Hide comment
@matsduf

matsduf Jun 25, 2016

Contributor

Update the specification of DELEGATION01 to make sure that the meaning is clear and cannot be missunderstood what "at lease two nameservers" mean.

Contributor

matsduf commented Jun 25, 2016

Update the specification of DELEGATION01 to make sure that the meaning is clear and cannot be missunderstood what "at lease two nameservers" mean.

@sandoche2k

This comment has been minimized.

Show comment
Hide comment
@sandoche2k

sandoche2k Nov 7, 2016

Contributor

Rather the modification should be done at DELEGATION 02 to check for a minimum of two IP addresses.

Contributor

sandoche2k commented Nov 7, 2016

Rather the modification should be done at DELEGATION 02 to check for a minimum of two IP addresses.

@sandoche2k

This comment has been minimized.

Show comment
Hide comment
@sandoche2k

sandoche2k Nov 7, 2016

Contributor

@vlevigneron updated the test specification zonemaster/zonemaster#478

Contributor

sandoche2k commented Nov 7, 2016

@vlevigneron updated the test specification zonemaster/zonemaster#478

matsduf added a commit that referenced this issue Nov 15, 2016

@sandoche2k sandoche2k added this to the 2017.1 milestone Nov 24, 2016

matsduf added a commit that referenced this issue Dec 5, 2016

Update install instructions (#250)
* Fix issues #140

* Updated certain text as well as the format for the shell commands

* Added a What to do Next section

* Added a What to do Next section

* Modified CPAN to CPANM and other minor changes

* Added missing close braces

* Modified the heading from instructions to install dependencies

* Modified cpanm to cpan. You need to install cpanm and also there are issues with cpanm while installing with sudo

* Update install doc based on review from mats

* Updated based on Mats feedback

* Grouped all instructions under the chosen OS. This was done due to the use of SU in FreeBSD

@sandoche2k sandoche2k modified the milestones: 2017.4, 2017.1 Sep 12, 2017

@sandoche2k

This comment has been minimized.

Show comment
Hide comment
@sandoche2k

sandoche2k Nov 27, 2017

Contributor

Update the error message from "Notice" to "Error" as discussed during the F2F on 27th Nov , 2017 @vlevigneron

Contributor

sandoche2k commented Nov 27, 2017

Update the error message from "Notice" to "Error" as discussed during the F2F on 27th Nov , 2017 @vlevigneron

@sandoche2k

This comment has been minimized.

Show comment
Hide comment
@sandoche2k

sandoche2k Dec 13, 2017

Contributor

@vlevigneron thanks for resolving this issue.

Contributor

sandoche2k commented Dec 13, 2017

@vlevigneron thanks for resolving this issue.

vlevigneron added a commit to vlevigneron/zonemaster-engine that referenced this issue Dec 13, 2017

vlevigneron added a commit that referenced this issue Dec 14, 2017

@sandoche2k

This comment has been minimized.

Show comment
Hide comment
@sandoche2k

sandoche2k Dec 14, 2017

Contributor

closed by PR #360

Contributor

sandoche2k commented Dec 14, 2017

closed by PR #360

@sandoche2k sandoche2k closed this Dec 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment