Skip to content

Add a system test to check that receiving multipart AXFR works as expected.#505

Open
ximon18 wants to merge 8 commits into
mainfrom
test-large-axfr-in
Open

Add a system test to check that receiving multipart AXFR works as expected.#505
ximon18 wants to merge 8 commits into
mainfrom
test-large-axfr-in

Conversation

@ximon18
Copy link
Copy Markdown
Member

@ximon18 ximon18 commented Mar 9, 2026

This test is currently passing.

The following error was seen sometimes when the PR was first created but has not been seen recently:

| 2026-03-09T14:36:39.039859Z  INFO refresh{zone=example.test source=Server { addr: 127.0.0.1:1055, tsig_key: None }}: cascaded::loader: Refreshing Name(example.test.)
| 2026-03-09T14:36:39.039887Z DEBUG refresh{zone=example.test source=Server { addr: 127.0.0.1:1055, tsig_key: None }}: cascaded::loader::server: Refreshing Name(example.test.) from server 127.0.0.1:1055
| 2026-03-09T14:36:39.039912Z DEBUG refresh{zone=example.test source=Server { addr: 127.0.0.1:1055, tsig_key: None }}: cascaded::loader::server: Attempting an IXFR against 127.0.0.1:1055 for Name(example.test.)
| 2026-03-09T14:36:39.040245Z ERROR refresh{zone=example.test source=Server { addr: 127.0.0.1:1055, tsig_key: None }}: cascaded::daemon: thread 'cascade-worker' (ProcessId(219), ThreadId(61)) panicked at src/loader/server.rs:231:14: internal error: entered unreachable code

The test could be improved, to actually detect multipart AXFR was handled (I verified manually by inserting some extra logging temporarily) and that the entirety of the large zone is received and published (currently it just does a line count check).

@ximon18 ximon18 added this to the 0.1.0-beta1 milestone Mar 9, 2026
@ximon18 ximon18 added the testing label Mar 9, 2026

# TODO: Verify that Cascade actually handled more than one AXFR part.
# Currently there is no NSD or Cascade metric that can be used to check
# this.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You should be able to use ZONEMD to let ldns-verify-zone check in the unsign review hook that the zone was received correctly.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants