Skip to content

Commit e498022

Browse files
test: Test ingest_email view
1 parent 89bb544 commit e498022

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

ietf/api/views.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -562,7 +562,7 @@ def __init__(
562562
self.msg = msg
563563
self.email_body = email_body
564564
self.email_subject = msg
565-
self.email_recipients = email_recipients or tuple(adm[1] for adm in settings.ADMINS)
565+
self.email_recipients = email_recipients
566566
self.email_attach_traceback = email_attach_traceback
567567
self.email_original_message = email_original_message
568568
self.email_from = settings.SERVER_EMAIL
@@ -578,15 +578,18 @@ def _summarize_error(error):
578578

579579
def as_emailmessage(self) -> Optional[EmailMessage]:
580580
"""Generate an EmailMessage to report an error"""
581-
if self.email_recipients is None:
582-
return None
581+
if self.email_body is None:
582+
return None
583583
error = self if self.__cause__ is None else self.__cause__
584584
format_values = dict(
585585
error=error,
586586
error_summary=self._summarize_error(error),
587587
)
588588
msg = EmailMessage()
589-
msg["To"] = self.email_recipients
589+
if self.email_recipients is None:
590+
msg["To"] = tuple(adm[1] for adm in settings.ADMINS)
591+
else:
592+
msg["To"] = self.email_recipients
590593
msg["From"] = self.email_from
591594
msg["Subject"] = self.msg
592595
msg.set_content(

0 commit comments

Comments
 (0)