Dry up MailHandler logging
Attached patch removes duplicated checks on logger presence in the MailHandler. Also, running "
if logger && logger.info" seems to be a dumb thing and adds spurious empty log lines.
The fix is to make
logger call always return a non-
nil object, even when logger is set to
nil in the environment configuration file (a default in production.) It might be good to include into Rails itself, so you never need to write another "
if logger" line.