summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSteve Kemp <steve@steve.org.uk>2013-04-25 13:09:04 +0100
committerSteve Kemp <steve@steve.org.uk>2013-04-25 13:09:04 +0100
commitff3ee93be58275fda1246c7d51ff63c4ccb69414 (patch)
tree07a8ee5b7009784602dfa8553663d2a9efc7fb10
parent3cf3942d6ef99a25ebac34952e6a7d151504f915 (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.rb8
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