Skip to content

Remove all MailHog references#7

Open
ryangriggs wants to merge 1 commit intomainfrom
claude/fix-auth-code-expiry-hHUOT
Open

Remove all MailHog references#7
ryangriggs wants to merge 1 commit intomainfrom
claude/fix-auth-code-expiry-hHUOT

Conversation

@ryangriggs
Copy link
Copy Markdown
Owner

Drops the MailHog dev-mail-catcher entirely. The app already listens on port 25 for inbound mail, so outbound dev testing can use the configured SMTP relay or Mailgun just like production.

  • src/config.js: remove mailhog config block
  • src/services/mail.js: remove isDev→MailHog transport branch; simplify send() condition
  • docker-compose.yml: remove mailhog service and app depends_on
  • docker-compose.prod.yml: remove stale comment
  • .env.example: remove MailHog variables and associated comments

https://claude.ai/code/session_01DmC97xUSF4zXH7BX3WRcFA

Drops the MailHog dev-mail-catcher entirely. The app already listens on
port 25 for inbound mail, so outbound dev testing can use the configured
SMTP relay or Mailgun just like production.

- src/config.js: remove mailhog config block
- src/services/mail.js: remove isDev→MailHog transport branch; simplify send() condition
- docker-compose.yml: remove mailhog service and app depends_on
- docker-compose.prod.yml: remove stale comment
- .env.example: remove MailHog variables and associated comments

https://claude.ai/code/session_01DmC97xUSF4zXH7BX3WRcFA
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.

2 participants