inbuxa/inbuxa-server
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 23 Wiki Activity

DNS test: expect the account-configuration digest inbuxa publishes #96

Merged
jcoffey-dev merged 1 commits from fix/dns-test-pacc-digest into main 2026-09-28 22:48:03 +00:00
Conversation 0 Commits 1 Files Changed 1
+4 -1
jcoffey-dev commented 2026-09-28 22:29:14 +00:00
Owner
Copy Link
Copy Source

automation::automation_tests failed at tests/src/automation/dns.rs:200 on main. It failed identically on main from before today's merges, with the same digest.

Cause. The expected zone was copied from upstream v0.16.22 (7dae9b2).

  • Its _ua-auto-config TXT record carries d= = SHA-256 of the account-configuration (PACC) document.
  • That document's provider name is types::brand!(), which the rebrand (89665de) changed.
  • So the server's digest (ZZ35…) is right; the test still expected upstream's (9X2m…).

Fix. Update the expected digest; notice header added to the file. Nothing else in the zone differed.

Checked on the laptop: with the ufw rule that lets the stalwart-test-acme network reach the host, the whole automation suite passes:

  • ACME, including #93's reschedule check (the first time it has run);
  • DKIM;
  • DNS;
  • RFC 2136.

notice-check passes.

`automation::automation_tests` failed at `tests/src/automation/dns.rs:200` on main. It failed identically on main from before today's merges, with the same digest. **Cause.** The expected zone was copied from upstream v0.16.22 (`7dae9b2`). - Its `_ua-auto-config` TXT record carries `d=` = SHA-256 of the account-configuration (PACC) document. - That document's provider name is `types::brand!()`, which the rebrand (`89665de`) changed. - So the server's digest (`ZZ35…`) is right; the test still expected upstream's (`9X2m…`). **Fix.** Update the expected digest; notice header added to the file. Nothing else in the zone differed. **Checked on the laptop:** with the ufw rule that lets the `stalwart-test-acme` network reach the host, the whole automation suite passes: - ACME, including #93's reschedule check (the first time it has run); - DKIM; - DNS; - RFC 2136. `notice-check` passes.
jcoffey-dev added 1 commit 2026-09-28 22:29:15 +00:00
DNS test: expect the account-configuration digest inbuxa publishes
ci / fork-checks (pull_request) Successful in 55s
Details
ci / build (pull_request) Successful in 18m46s
Details
11ba361c8c 
The automation suite's DNS test compared the published zone with one
copied from upstream v0.16.22. Its _ua-auto-config record carries a
SHA-256 of the account-configuration (PACC) document, and that document
names the provider as the brand, which the rebrand changed. The digest
the server publishes is right; the expected zone still held upstream's.

With the new digest the whole automation suite passes: ACME (including
the not-due reschedule check), DKIM, DNS and RFC 2136. It had been
failing at this point on main since the rebrand.
jcoffey-dev merged commit 0502eb45ed into main 2026-09-28 22:48:03 +00:00
jcoffey-dev deleted branch fix/dns-test-pacc-digest 2026-09-28 22:48:03 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
jcoffey-dev (John Coffey)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: inbuxa/inbuxa-server#96