From 9aaf507dd65083dd8e198470cefd9f4dac1704c5 Mon Sep 17 00:00:00 2001 From: Jasmine Schladen Date: Mon, 16 Nov 2020 17:36:59 -0800 Subject: [PATCH] Clarify comment --- lemur/notifications/messaging.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lemur/notifications/messaging.py b/lemur/notifications/messaging.py index 5f414b8c..2658e1a0 100644 --- a/lemur/notifications/messaging.py +++ b/lemur/notifications/messaging.py @@ -161,8 +161,10 @@ def send_expiration_notifications(exclude): success += len(email_recipients) else: failure += len(email_recipients) - # If we're using an email plugin, we're done; - # if not, we also need to send an email notification to the security team and owner + # If we're using an email plugin, we're done, + # since "security_email + [owner]" were added as email_recipients. + # If we're not using an email plugin, we also need to send an email to the security team and owner, + # since the plugin notification didn't send anything to them. if notification.plugin.slug != "email-notification": if send_default_notification( "expiration", notification_data, email_recipients, notification.options