Page MenuHomePhabricator

Use "cluster.mailers" if it is configured
ClosedPublic

Authored by epriestley on Feb 6 2018, 2:52 PM.
Tags
None
Referenced Files
F15438156: D19005.id45570.diff
Tue, Mar 25, 11:37 PM
F15436021: D19005.id45604.diff
Tue, Mar 25, 11:27 AM
F15431329: D19005.id.diff
Mon, Mar 24, 11:46 AM
F15398963: D19005.id45570.diff
Mon, Mar 17, 2:04 AM
F15393951: D19005.id45604.diff
Sat, Mar 15, 11:18 PM
F15393948: D19005.diff
Sat, Mar 15, 11:18 PM
F15357354: D19005.id45604.diff
Mar 11 2025, 6:41 AM
Unknown Object (File)
Mar 3 2025, 1:01 PM
Subscribers
None

Details

Summary

Depends on D19004. Ref T13053. Ref T12677. If the new cluster.mailers is configured, make use of it. Also use it in the Sengrid/Mailgun inbound stuff.

Also fix a bug where "Must Encrypt" mail to no recipients could fatal because no $mail was returned.

Test Plan

Processed some mail locally. The testing on this is still pretty flimsy, but I plan to solidify it in an upcoming change.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley added inline comments.
src/applications/metamta/storage/PhabricatorMetaMTAMail.php
485

This works because we don't invoke validateOptions() before invoking setOptions(), right?

This revision is now accepted and ready to land.Feb 7 2018, 8:43 PM
src/applications/metamta/storage/PhabricatorMetaMTAMail.php
485

The top-level shared stuff (key, type, priority) is a level above options. setOptions() will validate options, but "options" only has mailer-specifc stuff (api key, domain, username, etc.), not shared stuff.

This revision was automatically updated to reflect the committed changes.