Email notifier: support sending mail through sendmail
Reported by Simon Rozet | May 19th, 2009 @ 12:12 PM
sinatra-mailer already supports it. We need to modify the configuration form to exposes the sendmail option.
diff --git a/lib/integrity/notifier/email.rb b/lib/integrity/notifier/email.rb
index 04a8829..92199cd 100644
--- a/lib/integrity/notifier/email.rb
+++ b/lib/integrity/notifier/email.rb
@@ -38,21 +38,8 @@ module Integrity
private
def configure_mailer
- user = @config["user"] || ""
- pass = @config["pass"] || ""
- user = nil if user.empty?
- pass = nil if pass.empty?
-
- Sinatra::Mailer.delivery_method = "net_smtp"
-
- Sinatra::Mailer.config = {
- :host => @config["host"],
- :port => @config["port"],
- :user => user,
- :pass => pass,
- :auth => @config["auth"],
- :domain => @config["domain"]
- }
+ Sinatra::Mailer.delivery_method = :sendmail
+ Sinatra::Mailer.config[:sendmail_path] = "/usr/sbin/sendmail"
end
end
Comments and changes to this ticket
-
Simon Rozet June 3rd, 2009 @ 06:18 PM
- State changed from “new” to “resolved”
- Assigned user set to “Simon Rozet”
Thanks to David Reese (http://github.com/whatcould/integrity-email/commit/1e9207f19640b3da...) that is fixed in integrity-email 1.1.4
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
Automated continuous integration server that doesn't suck.
Simon Rozet