Description:
update mail functionality. Update syntax for mail gem (we might need more options for mail setting?)
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r348:fc78be8c614c - - 2 files changed: 14 inserted, 3 deleted
@@ -126,13 +126,13 | |||
|
126 | 126 | mail_body = t('registration.email_body', { |
|
127 | 127 | :full_name => user.full_name, |
|
128 | 128 | :contest_name => contest_name, |
|
129 | 129 | :login => user.login, |
|
130 | 130 | :password => user.password, |
|
131 | 131 | :activation_url => activation_url, |
|
132 | - :admin_email => admin_email | |
|
132 | + :admin_email => GraderConfiguration['system.admin_email'] | |
|
133 | 133 | }) |
|
134 | 134 | |
|
135 | 135 | logger.info mail_body |
|
136 | 136 | |
|
137 | 137 | send_mail(user.email, mail_subject, mail_body) |
|
138 | 138 | end |
@@ -142,13 +142,13 | |||
|
142 | 142 | mail_subject = "[#{contest_name}] Password recovery" |
|
143 | 143 | mail_body = t('registration.password_retrieval.email_body', { |
|
144 | 144 | :full_name => user.full_name, |
|
145 | 145 | :contest_name => contest_name, |
|
146 | 146 | :login => user.login, |
|
147 | 147 | :password => user.password, |
|
148 | - :admin_email => admin_email | |
|
148 | + :admin_email => GraderConfiguration['system.admin_email'] | |
|
149 | 149 | }) |
|
150 | 150 | |
|
151 | 151 | logger.info mail_body |
|
152 | 152 | |
|
153 | 153 | send_mail(user.email, mail_subject, mail_body) |
|
154 | 154 | end |
@@ -19,12 +19,23 | |||
|
19 | 19 | from mail_from |
|
20 | 20 | to mail_to |
|
21 | 21 | subject mail_subject |
|
22 | 22 | body mail_body |
|
23 | 23 | end |
|
24 | 24 | |
|
25 | - mail.delivery_settings = { :address => smtp_server } | |
|
25 | + mail_option = { | |
|
26 | + :address => smtp_server, | |
|
27 | + # :domain => nil, | |
|
28 | + # :port => 25, | |
|
29 | + # :user_name => nil, | |
|
30 | + # :password => nil, | |
|
31 | + # :authentication=>'plain', | |
|
32 | + # :enable_starttls_auto => true | |
|
33 | + } | |
|
34 | + | |
|
35 | + mail.delivery_method :smtp, mail_option | |
|
36 | + | |
|
26 | 37 | mail.deliver |
|
27 | 38 | end |
|
28 | 39 | |
|
29 | 40 | end |
|
30 | 41 |
You need to be logged in to leave comments.
Login now