HomePhabricator
Diffusion Comm 7ff4f9482a74

[keyserver] Use SMTP (via Postmark) instead of `sendmail` for email…

Description

[keyserver] Use SMTP (via Postmark) instead of sendmail for email subscriptions

Summary:
The SubscriptionForm on the landing page broke when we moved the keyserver to EC2 because the image didn't include sendmail. After installing sendmail it appeared that things were working and the requests were succeeding, but the emails didn't end up getting delivered reliably. There's additional configuration that's necessary to get the sendmail approach working reliably/consistently.

Instead, in this diff we move from the sendmail transport to using SMTP via nodemailer with Postmark as the "host."

These are some resources I referenced to possibly save reviewer some googling:

Test Plan:

  1. Change the to field in sendmail.sendMail to my Comm email account
  2. Navigate to localhost/commlanding/
  3. Put a valid email in the SubscriptionForm input
  4. Click "Subscribe for updates"
  5. Observe that the email shows up in Postmark
  6. Ensure that the email showed up in my inbox

Reviewers: def-au1t, palys-swm, yayabosh, ashoat

Reviewed By: ashoat

Subscribers: Adrian

Differential Revision: https://phab.comm.dev/D4291

Details

Provenance
atulAuthored on Jul 5 2022, 6:23 AM
Reviewer
ashoat
Differential Revision
D4291: [keyserver] Use SMTP (via Postmark) instead of `sendmail` for email subscriptions
Parents
rCOMM78751dfb1334: [services] Clean up `test_all_services.sh`
Branches
Unknown
Tags
Unknown