Raise SSL errors and respect action mailer's :enable_starttls_auto setting #10

Open
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants

Hi,

I ran into problems using ar_sendmail with a mail server that can use tls, but doesn't have a valid certificate, and made a couple of changes to the code as a result...

SSL errors are no longer silently ignored, and action mailer's new(ish) :enable_starttls_auto is respected within ActionMailer::ARSendmail#deliver

These changes seem sensible to me, you may wish to pull them in if you agree.

Mark

I ran into the exact same issue. I'd like to see this fix merged too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment