From 14ef3dcce18240b756415fefa2a56936f96a12e9 Mon Sep 17 00:00:00 2001 From: drebs Date: Mon, 13 Apr 2015 15:58:16 -0300 Subject: [bug] fix bounce message recipient The bounce message was using the original message's "From:" header instead of the "To:" header to indicate the original recipient. This commit fixes that. Closes: #6854. Releases: 0.6.2, 0.7.0 --- changes/bug_6854_fix-bounce-message-recipient | 1 + src/leap/mx/mail_receiver.py | 7 ++++--- 2 files changed, 5 insertions(+), 3 deletions(-) create mode 100644 changes/bug_6854_fix-bounce-message-recipient diff --git a/changes/bug_6854_fix-bounce-message-recipient b/changes/bug_6854_fix-bounce-message-recipient new file mode 100644 index 0000000..417c894 --- /dev/null +++ b/changes/bug_6854_fix-bounce-message-recipient @@ -0,0 +1 @@ + o Fix bounce message recipient. Closes #6854. diff --git a/src/leap/mx/mail_receiver.py b/src/leap/mx/mail_receiver.py index 7856594..77909b0 100644 --- a/src/leap/mx/mail_receiver.py +++ b/src/leap/mx/mail_receiver.py @@ -392,15 +392,16 @@ class MailReceiver(Service): :param reason: Brief explanation about why it's being bounced :type reason: str """ - to = orig_msg.get("From") + orig_from = orig_msg.get("From") + orig_to = orig_msg.get("To") msg = MIMEMultipart() msg['From'] = self._bounce_from - msg['To'] = to + msg['To'] = orig_from msg['Date'] = formatdate(localtime=True) msg['Subject'] = self._bounce_subject - decoded_to = " ".join([x[0] for x in decode_header(to)]) + decoded_to = " ".join([x[0] for x in decode_header(orig_to)]) text = BOUNCE_TEMPLATE.format(decoded_to, reason, orig_msg.as_string()) -- cgit v1.2.3