Skip to content

Conversation

@tgreenx
Copy link
Contributor

@tgreenx tgreenx commented Dec 9, 2025

Purpose

This PR updates DNSSEC07 message IDs as specified in zonemaster/zonemaster#1449.

Context

zonemaster/zonemaster#1449

Changes

  • Message IDs

How to test this PR

  1. Unit tests should still pass.
  2. Check that the updates match the ones in Update DNSSEC07 message IDs zonemaster#1449.
  3. Run DNSSEC07 on a signed zone and check the output, e.g. with zonemaster-cli:
$ zonemaster-cli --show-testcase --level INFO --test dnssec07 --no-ipv6 afnic.fr

Seconds Level    Testcase       Message
======= ======== ============== =======
   0.00 INFO     Unspecified    Using version v8.0.0 of the Zonemaster engine.
  12.96 INFO     DNSSEC07       The following name servers respond with DNSKEY (signed child zone). Name servers: "g.ext.nic.fr/194.0.36.1;ns1.nic.fr/192.134.4.1;ns2.nic.fr/192.93.0.4;ns3.nic.fr/192.134.0.49".
  12.96 INFO     DNSSEC07       The zone is signed.
  12.96 INFO     DNSSEC07       The following parent name servers respond with DS record or records for the child zone. Name servers: "d.nic.fr/194.0.9.1;f.ext.nic.fr/194.146.106.46;g.ext.nic.fr/194.0.36.1".
  12.96 INFO     DNSSEC07       The parent zone has DS record or records for the signed child zone.

@tgreenx tgreenx added this to the v2025.2 milestone Dec 9, 2025
@tgreenx tgreenx requested a review from matsduf December 9, 2025 15:43
@tgreenx tgreenx added A-TestCase Area: Test case specification or implementation of test case V-Patch Versioning: The change gives an update of patch in version. RC-None Release category: Not to be included in Changes file. labels Dec 9, 2025
@tgreenx tgreenx merged commit bfcee6a into zonemaster:develop Dec 9, 2025
3 checks passed
@tgreenx tgreenx deleted the update-dnssec07-msg branch December 9, 2025 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-TestCase Area: Test case specification or implementation of test case RC-None Release category: Not to be included in Changes file. V-Patch Versioning: The change gives an update of patch in version.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants