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 /lib/custodian/alerts/smtp.rb | |
parent | 3cf3942d6ef99a25ebac34952e6a7d151504f915 (diff) |
Correct the body of emails on raise()/clear().
Credit to reddit user knothead for spotting this bug.
Fixes #4386.
Diffstat (limited to 'lib/custodian/alerts/smtp.rb')
-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 |