Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/discourse_docker/templates/app.yml.erb

Issue 29427603: Noissue - Configure discourse containers to use smtp on the host system Base URL: https://hg1/infrastructure
Patch Set: Created May 2, 2017, 9:23 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: modules/discourse_docker/templates/app.yml.erb
===================================================================
--- a/modules/discourse_docker/templates/app.yml.erb
+++ b/modules/discourse_docker/templates/app.yml.erb
@@ -50,7 +50,7 @@
DISCOURSE_DEVELOPER_EMAILS: '<% @admins.each do |admin| %><%= admin %>,<% end %>'
## TODO: The SMTP mail server used to validate new accounts and send notifications
- DISCOURSE_SMTP_ADDRESS: localhost # required
+ DISCOURSE_SMTP_ADDRESS: <%= @smtp_host %> # required
DISCOURSE_SMTP_PORT: 25 # (optional, default 587)
#DISCOURSE_SMTP_USER_NAME: user@example.com # required
#DISCOURSE_SMTP_PASSWORD: pa$$word # required, WARNING the char '#' in pw can cause problems!

Powered by Google App Engine
This is Rietveld