From cde99c668fc99e67c8c4db10fff161b4dd4b2530 Mon Sep 17 00:00:00 2001 From: Steve Kemp Date: Thu, 25 Apr 2013 13:09:04 +0100 Subject: Correct the body of emails on raise()/clear(). Credit to reddit user knothead for spotting this bug. Fixes #4386. --- lib/custodian/alerts/smtp.rb | 8 ++++---- 1 file 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 -- cgit v1.2.1