diff options
| author | Steve Kemp <steve@steve.org.uk> | 2013-04-25 13:09:04 +0100 | 
|---|---|---|
| committer | Steve Kemp <steve@steve.org.uk> | 2013-04-25 13:09:04 +0100 | 
| commit | ff3ee93be58275fda1246c7d51ff63c4ccb69414 (patch) | |
| tree | 07a8ee5b7009784602dfa8553663d2a9efc7fb10 | |
| parent | 3cf3942d6ef99a25ebac34952e6a7d151504f915 (diff) | |
  Correct the body of emails on raise()/clear().
  Credit to reddit user knothead for spotting this bug.
  Fixes #4386.
| -rw-r--r-- | lib/custodian/alerts/smtp.rb | 8 | 
1 files changed, 4 insertions, 4 deletions
| diff --git a/lib/custodian/alerts/smtp.rb b/lib/custodian/alerts/smtp.rb index d170bbf..4de6fbe 100644 --- a/lib/custodian/alerts/smtp.rb +++ b/lib/custodian/alerts/smtp.rb @@ -30,8 +30,8 @@ module Custodian        # Raise an alert by email.        #        def raise -        subject = "#{test.target} failed #{test.get_type}-test - #{test.error()}" -        body    = "The alert has cleared\nRegards\n"; +        subject = "#{test.target} alert #{test.get_type}-test - #{test.error()}" +        body    = "The alert has raised, with the following details:\n#{test.error()}\nRegards\n";          _send_mail( @target, subject, body )        end @@ -42,8 +42,8 @@ module Custodian        # Clear an alert by email.        #        def clear -        subject = "#{test.target} failed #{test.get_type}-test" -        body    = "The alert has raised, with the following details:\n#{test.error()}\nRegards\n"; +        subject = "#{test.target} cleared #{test.get_type}-test" +        body    = "The alert has cleared\nRegards\n";          _send_mail( @target, subject, body )        end | 
