New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document how to use symfony/mailer for email SMTP relay #844
base: 3.x
Are you sure you want to change the base?
Conversation
c62f131
to
3dd7f99
Compare
I came across this core issue, which is harmless but causes errors to appear in Apache logs: https://www.drupal.org/project/drupal/issues/3420372 There's a patch available, but I don't think we need to include it in farmOS. Hopefully it will be fixed in core soon. |
…rror on transport shutdown
On second thought, it's easy enough for us to include a patch, and risk is very low because it only touches the I pushed a second commit to add this patch. |
@paul121 and I isolated another Drupal core issue with the I'm going to look into providing a patch for it... |
…xception when sending to multiple comma-separated addresses
Seeing these various bugs come up and considering that Drupal core still doesn't consider |
I tend to agree we should wait on this PR. In the meantime I will test it out in Farmier and help to debug issues. Hopefully these two obvious bugs will be fixed in core quick so no new Drupal core patches will ever be needed in farmOS.
That said, I think there's a difference between the "complete" integration as outlined in https://www.drupal.org/project/drupal/issues/1803948 and the simpler "SMTP relay" functionality that we currently recommend using the SMTP module for. The change record (https://www.drupal.org/node/3369935) refers to the new |
This changes our https://farmos.org/hosting/email/ documentation to recommend using
symfony/mailer
for SMTP email relays, instead of the contrib SMTP module.Kudos to @pcambra for turning me on to this new option! #837 (comment)