For places like handling of IPN web hook we are using non-queued mail, which could cause failures in processing if there are failures when sending email, which is not good.
In addition, there is a fair amount of complexity in tests surrounding testing for queued mail.
We could fix these by using queued mail always, or at least by default. We might want to avoid queued mail for:
sending magic login emails - if email sending fails, it's actually better to get a 500 so the user knows the email never got sent
for mailing lists, where we are just forwarding emails and might not want to store them in the DB at all.
For places like handling of IPN web hook we are using non-queued mail, which could cause failures in processing if there are failures when sending email, which is not good.
In addition, there is a fair amount of complexity in tests surrounding testing for queued mail.
We could fix these by using queued mail always, or at least by default. We might want to avoid queued mail for: