SMTP_ENABLE_STARTTLS_AUTO should be a boolean
Closed this issue · 1 comments
Hi! I'm trying to setup ciao on a homelab kubernetes cluster - where I already have an in-house open mail relay that's just doing doing raw port 25, no-ssl (and in turn relays to an actual authenticated provider).
As far as I'm aware, the Action Mailer 'enable_starttls_auto' should be a boolean value - the docs indicate 'auto' which I don't think is valid (and thereby it just defaults to true in action_mailer). Setting it to false in the environment values (or helm values) just has it end up as a "false" string in the actual rails configuration.
I'm guessing it needs the same ActiveModel::Type::Boolean.new.cast casting that ENV['SMTP_SSL'] received.
I can submit a PR - but haven't set up a ciao development environment - and figured this might just be easier as a quick issue report.
@jasonadamyoung Thanks for reporting.
Fixed and updated the README.