#158 ✓resolved
Simon Rozet

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

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.

New-ticket 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.

Shared Ticket Bins

People watching this ticket

Pages