diff options
author | Brian Durand <brian@embellishedvisions.com> | 2012-09-19 16:00:28 -0700 |
---|---|---|
committer | Brian Durand <brian@embellishedvisions.com> | 2012-09-30 09:47:24 -0700 |
commit | 37852726c93541aa1adc014cfb862659c9bcf349 (patch) | |
tree | 06cc8369010e8f3bd201cf54d9217e0223021248 /actionmailer | |
parent | 01059d7f7fcf2780c425895e4bdd5395edb6436b (diff) | |
download | rails-37852726c93541aa1adc014cfb862659c9bcf349.tar.gz rails-37852726c93541aa1adc014cfb862659c9bcf349.tar.bz2 rails-37852726c93541aa1adc014cfb862659c9bcf349.zip |
Optimize log subscribers to check if the log level is sufficient before performing an operations.
Diffstat (limited to 'actionmailer')
-rw-r--r-- | actionmailer/lib/action_mailer/log_subscriber.rb | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/actionmailer/lib/action_mailer/log_subscriber.rb b/actionmailer/lib/action_mailer/log_subscriber.rb index a6c163832e..3fe64759ac 100644 --- a/actionmailer/lib/action_mailer/log_subscriber.rb +++ b/actionmailer/lib/action_mailer/log_subscriber.rb @@ -1,13 +1,15 @@ module ActionMailer class LogSubscriber < ActiveSupport::LogSubscriber def deliver(event) + return unless logger.info? recipients = Array(event.payload[:to]).join(', ') - info("\nSent mail to #{recipients} (%1.fms)" % event.duration) + info("\nSent mail to #{recipients} (#{event.duration.round(1)}ms)") debug(event.payload[:mail]) end def receive(event) - info("\nReceived mail (%.1fms)" % event.duration) + return unless logger.info? + info("\nReceived mail (#{event.duration.round(1)}ms)") debug(event.payload[:mail]) end |