Skip to content

fix(mail): allow cascading additional domains - #77

Merged
Hydralerne merged 2 commits into
oblien:mainfrom
Rajioba1:fix/additional-domain-cascade-delete
Jul 21, 2026
Merged

fix(mail): allow cascading additional domains#77
Hydralerne merged 2 commits into
oblien:mainfrom
Rajioba1:fix/additional-domain-cascade-delete

Conversation

@Rajioba1

Copy link
Copy Markdown
Contributor

Summary

  • scope postmaster hard-delete protection to the primary install domain
  • allow automatically-created postmaster mailboxes on additional domains to be removed during cascade deletion
  • fail closed when the install domain cannot be read
  • add regression coverage for additional, primary, and unavailable-state cases

Verification

  • bunx vitest run test/modules/mail/mailboxes.service.test.ts test/modules/mail/dns-scan.service.test.ts test/modules/mail/psql-injection.test.ts (11 tests passed)
  • bun run --cwd apps/api lint
  • Prettier check on both touched files
  • git diff --check

Formatting note

mailboxes.service.ts was not Prettier-clean on main. Per CONTRIBUTING.md, the formatting-only changes are isolated in the first commit; the functional change and tests are in the second commit.

Fixes #76

@Hydralerne
Hydralerne merged commit 173b642 into oblien:main Jul 21, 2026
1 check passed
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.

bug(mail): additional-domain cascade deletion fails on postmaster mailbox

2 participants