Fix queries and response packets content for undelegated tests

[tgreenx]

Purpose

For undelegated tests (fake delegation), this PR adds the QR flag in queries and AA flag depending if DS record is provided or not.

Context

Fixes #1100

Changes

• Sets the QR flag for queries in undelegated tests
• Sets the AA flag for queries in undelegated tests with DS record
• Returns empty sections in response packet to a DS query in undelegated tests with no DS record

How to test this PR

zonemaster-cli 190.187.193.in-addr.arpa --ns ns1.oxilion.nl --ns ns2.oxilion.nl --ns ns3.oxilion.net --raw --level debug3 --test dnssec/dnssec11 > debug-dnssec11

Inspect debug file debug-dnssec11. Search for type=DS. Look at flags and packet content thereafter. You can change query type in dnssec11 and compare.

[matsduf]

Inspect debug file debug-dnssec11. Search for type=DS. Look at flags and packet content thereafter. You can change query type in dnssec11 and compare.

How can you change the query type when running a test case?

[tgreenx]

Inspect debug file debug-dnssec11. Search for type=DS. Look at flags and packet content thereafter. You can change query type in dnssec11 and compare.

How can you change the query type when running a test case?

You change query type directly in the code of the Test Case, for example here. And just add die; right after to abort the test early, it is enough for our purpose.

my $ds_p = $ns->query( $zone->name, q{SOA}, { dnssec => 1, usevc => 0 } );
die;

[tgreenx]

@matsduf please re-review

[matsduf]

The "How to test" does not work. In undelegated test there is no DS query for DNSSEC11. The following command works better:

zonemaster-cli 190.187.193.in-addr.arpa --ns ns1.oxilion.nl --ns ns2.oxilion.nl --ns ns3.oxilion.net --raw --level debug3 --test dnssec/dnssec02 > debug-dnssec02

[matsduf]

The result looks strange, but it probably does not affect the end result of Zonemaster. Issue #1169 has been created.