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

DNSSEC16 test case implementation #900

Merged

Conversation

vlevigneron
Copy link
Contributor

@vlevigneron vlevigneron commented May 11, 2021

  • The code is now ready to be merged
  • Only one message tag not tested for the moment (should be added on a later PR).

Updated 2021-05-17:

How to test this PR

Read the test case specification for DNSSEC16 and verify that the implementation correctly reports issues and non-issues on selected zones to cover as many message tags as possible.

@matsduf matsduf added this to the v2021.1 milestone May 11, 2021
@vlevigneron vlevigneron requested a review from matsduf May 11, 2021 20:03
@vlevigneron
Copy link
Contributor Author

@matsduf The messages are not yet fully implemented but this test case can already be tested.

@vlevigneron
Copy link
Contributor Author

@matsduf implementation "finished" (no translation yet). We have to add new unit tests when we will find broken zone or when we will be able to configure these broken zone.

@vlevigneron
Copy link
Contributor Author

@matsduf Can you have a look ? I will not add more unit tests (only one message tage is not covered for the moment, I made some modification on Bind to prevent some checks that allow to load broken zones). I start to work on dnssec17. If we can merge dnssec16 before, it will be easier for the merge.

Copy link
Contributor

@matsduf matsduf left a comment

Choose a reason for hiding this comment

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

I will do some "live" tests too.

lib/Zonemaster/Engine/Test/DNSSEC.pm Outdated Show resolved Hide resolved
lib/Zonemaster/Engine/Test/DNSSEC.pm Outdated Show resolved Hide resolved
lib/Zonemaster/Engine/Test/DNSSEC.pm Outdated Show resolved Hide resolved
lib/Zonemaster/Engine/Test/DNSSEC.pm Outdated Show resolved Hide resolved
lib/Zonemaster/Engine/Test/DNSSEC.pm Outdated Show resolved Hide resolved
lib/Zonemaster/Engine/Test/DNSSEC.pm Outdated Show resolved Hide resolved
lib/Zonemaster/Engine/Test/DNSSEC.pm Outdated Show resolved Hide resolved
lib/Zonemaster/Engine/Test/DNSSEC.pm Outdated Show resolved Hide resolved
@vlevigneron vlevigneron requested a review from matsduf May 17, 2021 09:36
lib/Zonemaster/Engine/Test/DNSSEC.pm Show resolved Hide resolved
t/Test-dnssec.t Show resolved Hide resolved
t/Test-dnssec.t Show resolved Hide resolved
@vlevigneron
Copy link
Contributor Author

@matsduf Can you test again with dnssec16-delete-cds.zft-root.rd.nic.fr broken zone ? I do not understand why you have not the DS16_DELETE_CDS message tag.

@vlevigneron vlevigneron requested a review from matsduf May 17, 2021 13:13
@matsduf
Copy link
Contributor

matsduf commented May 17, 2021

@matsduf Can you test again with dnssec16-delete-cds.zft-root.rd.nic.fr broken zone ? I do not understand why you have not the DS16_DELETE_CDS message tag.

# zonemaster-cli --no-ipv6 --show_testcase --test DNSSEC dnssec16-delete-cds.zft-root.rd.nic.fr --locale en_US.UTF-8
Seconds Level     Testcase       Message
======= ========= ============== =======
   4.93 WARNING   DNSSEC05       The DNSKEY with tag 64275 uses an algorithm number 5 (RSA/SHA1) which is not recommended to be used.
   4.93 WARNING   DNSSEC05       The DNSKEY with tag 37774 uses an algorithm number 5 (RSA/SHA1) which is not recommended to be used.
   4.94 WARNING   DNSSEC05       The DNSKEY with tag 37774 uses an algorithm number 5 (RSA/SHA1) which is not recommended to be used.
   4.94 WARNING   DNSSEC05       The DNSKEY with tag 64275 uses an algorithm number 5 (RSA/SHA1) which is not recommended to be used.
   5.23 WARNING   DNSSEC14       DNSKEY with tag 64275 and using algorithm 5 (RSA/SHA1) has a size (1024) smaller than the recommended one (2048).
   5.40 NOTICE    DNSSEC15       CDS RRset is found on nameservers that resolve to IP addresses (145.239.76.199;145.239.76.201), but no CDNSKEY RRset.

@vlevigneron
Copy link
Contributor Author

@matsduf that's because you should add "--level info" to see this kind (INFO) of message tag.

@matsduf
Copy link
Contributor

matsduf commented May 17, 2021

@matsduf that's because you should add "--level info" to see this kind (INFO) of message tag.

Correct. Missed that.

@vlevigneron vlevigneron merged commit 0146f77 into zonemaster:develop May 17, 2021
@matsduf
Copy link
Contributor

matsduf commented May 30, 2021

Release testing for v2021.1

Works as expected.

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