Skip to content

fix(backend): log mail delivery failures - #600

Open
dollarop wants to merge 1 commit into
Protocol-Guild:mainfrom
dollarop:fix/mailer-failure-logging-454
Open

fix(backend): log mail delivery failures#600
dollarop wants to merge 1 commit into
Protocol-Guild:mainfrom
dollarop:fix/mailer-failure-logging-454

Conversation

@dollarop

Copy link
Copy Markdown

Summary

  • log a warning when SMTP configuration is missing
  • log nodemailer import failures and delivery failures with recipient and mail type context
  • avoid logging SMTP credentials or message content
  • add regression tests for previously silent failure paths

Closes #454

Verification

  • npm test -- --runInBand src/services/__tests__/mailerService.test.ts (2 tests passed)
  • npm run build remains blocked by pre-existing syntax errors in backend/src/services/tenantConfigService.ts:207 and nearby lines

@dollarop
dollarop marked this pull request as ready for review August 23, 2026 04:59
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.

MailerService silently no-ops on SMTP misconfiguration with no logging

1 participant