Fix handling of non-ascii in realname in the nosy mailer...

...this used to mangle the email address making it unusable when
replying. Thanks to intevation for funding the fix.

Author: Ralf Schlatterbeck

CHANGES.txt

@@ -10,6 +10,9 @@ Fixes:
   http://thread.gmane.org/gmane.comp.bug-tracking.roundup.devel/5133
   Add regression tests for proper handling of 'Create' and 'Edit'
   permissions.
+- Fix handling of non-ascii in realname in the nosy mailer, this used to
+  mangle the email address making it unusable when replying. Thanks to
+  intevation for funding the fix.
 
 2009-12-21 1.4.11 (r4411)
 

roundup/mailer.py

@@ -61,10 +61,15 @@ def get_standard_message(self, to, subject, author=None, multipart=False):
         charset = getattr(self.config, 'EMAIL_CHARSET', 'utf-8')
         tracker_name = unicode(self.config.TRACKER_NAME, 'utf-8')
         if not author:
-            author = formataddr((tracker_name, self.config.ADMIN_EMAIL))
+            author = (tracker_name, self.config.ADMIN_EMAIL)
+            name = author[0]
         else:
             name = unicode(author[0], 'utf-8')
-            author = formataddr((name, author[1]))
+        try:
+            name = name.encode('ascii')
+        except UnicodeError:
+            name = Header(name, charset).encode()
+        author = formataddr((name, author[1]))
 
         if multipart:
             message = MIMEMultipart()
@@ -77,10 +82,7 @@ def get_standard_message(self, to, subject, author=None, multipart=False):
         except UnicodeError:
             message['Subject'] = Header(subject, charset)
         message['To'] = ', '.join(to)
-        try:
-            message['From'] = author.encode('ascii')
-        except UnicodeError:
-            message['From'] = Header(author, charset)
+        message['From'] = author
         message['Date'] = formatdate(localtime=True)
 
         # add a Precedence header so autoresponders ignore us

test/test_mailgw.py

@@ -1181,6 +1181,9 @@ def testUnknownUser(self):
 ''')
 
     def testEnc01(self):
+        self.db.user.set(self.mary_id,
+            realname='\xe4\xf6\xfc\xc4\xd6\xdc\xdf, Mary'.decode
+            ('latin-1').encode('utf-8'))
         self.doNewIssue()
         self._handle_mail('''Content-Type: text/plain;
   charset="iso-8859-1"
@@ -1202,7 +1205,8 @@ def testEnc01(self):
 Content-Type: text/plain; charset="utf-8"
 Subject: [issue1] Testing...
 To: [email protected], [email protected]
-From: "Contrary, Mary" <[email protected]>
+From: =?utf-8?b?w6TDtsO8w4TDlsOcw58sIE1hcnk=?=
+ <[email protected]>
 Reply-To: Roundup issue tracker <[email protected]>
 MIME-Version: 1.0
 Message-Id: <followup_dummy_id>
@@ -1213,7 +1217,8 @@ def testEnc01(self):
 Content-Transfer-Encoding: quoted-printable
 
 
-Contrary, Mary <[email protected]> added the comment:
+=C3=A4=C3=B6=C3=BC=C3=84=C3=96=C3=9C=C3=9F, Mary <[email protected]> added the=
+ comment:
 
 A message with encoding (encoded oe =C3=B6)