Skip to content

Commit e904041

Browse files
committed
Don't try to do string join on None, in case any address in an address list given to the mail sending routines is missing and represented by None.
- Legacy-Id: 3111
1 parent cf10bc0 commit e904041

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

ietf/utils/mail.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -157,9 +157,9 @@ def send_mail_mime(request, to, frm, subject, msg, cc=None, extra=None, toUser=N
157157
if isinstance(frm, tuple):
158158
frm = formataddr(frm)
159159
if isinstance(to, list) or isinstance(to, tuple):
160-
to = ", ".join([isinstance(addr, tuple) and formataddr(addr) or addr for addr in to])
160+
to = ", ".join([isinstance(addr, tuple) and formataddr(addr) or addr for addr in to if addr])
161161
if isinstance(cc, list) or isinstance(cc, tuple):
162-
cc = ", ".join([isinstance(addr, tuple) and formataddr(addr) or addr for addr in cc])
162+
cc = ", ".join([isinstance(addr, tuple) and formataddr(addr) or addr for addr in cc if addr])
163163
if frm:
164164
msg['From'] = frm
165165
msg['To'] = to

0 commit comments

Comments
 (0)