Infra/lemur
5
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Clarify comment

Browse Source
This commit is contained in:
Jasmine Schladen 2020-11-16 17:36:59 -08:00
parent d22156337a
commit 9aaf507dd6
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lemur/notifications/messaging.py
View File

@ -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