-
Notifications
You must be signed in to change notification settings - Fork 203
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
Do we want to support Email/Contact? #15
Comments
See docker-library/wordpress#30 for a very closely related discussion. The TL;DR is that it's hard to configure this in such a way that it's sane-by-default for most use cases, so I think documenting how to configure it might be a safer solution overall. |
Documenting a good way to get around the issue would be great, as this function is also needed for password recovery for example... |
Or documenting on how to link your Drupal container to an mail container. |
After failing to link the |
Simple solution - extend the Drupal image with a custom Dockerfile and add SSMTP: Dockerfile:
I also add configuration to SSMTP to relay through AWS SES or Mandrill, but I think the above works without anything extra. |
I don't think there's a good generic solution we can apply here, but https://www.drupal.org/docs/develop/local-server-setup/managing-mail-handling-for-development-or-testing does have some good ideas, including pointers to https://www.drupal.org/project/smtp and https://www.drupal.org/project/phpmailer_smtp, one of which is definitely what I'd be using to solve this if I were running Drupal in my environment (either should be able to SMTP to a sidecar exim4 or otherwise container just as well as it can SMTP to an external provider). Another option is something like |
Agreed. I am closing this issue. For reference, if you want mailhog to intercept all emails, you can install mhsendmail |
Both D7 and D8 ship with Contact Module, which generates a web form and send the inquiry to Emails. With this image, this function doesn't work, because there is no email server included in the image.
So, I am wondering if we want to support this Core function in this image?
The text was updated successfully, but these errors were encountered: