Merge r117102 from trunk; encode usernames and passwords to utf-8 if …

…unicode

CHANGES.txt

@@ -4,6 +4,9 @@ Change history
 3.5.2 (unreleased)
 ------------------
 
+- Handle unicode usernames and passwords, encoding them to UTF-8. Fix for
+  https://bugs.launchpad.net/zope.sendmail/+bug/597143
+
 
 3.5.1 (2009-01-26)
 ------------------

src/zope/sendmail/mailer.py

@@ -64,7 +64,12 @@ def send(self, fromaddr, toaddrs, message):
 
         if connection.does_esmtp:
             if self.username is not None and self.password is not None:
-                connection.login(self.username, self.password)
+                username, password = self.username, self.password
+                if isinstance(username, unicode):
+                    username = username.encode('utf-8')
+                if isinstance(password, unicode):
+                    password = password.encode('utf-8')
+                connection.login(username, password)
         elif self.username:
             raise RuntimeError('Mailhost does not support ESMTP but a username '
                                 'is configured')

src/zope/sendmail/tests/test_mailer.py

@@ -112,6 +112,30 @@ def test_send_auth(self):
         self.assert_(self.smtp.quitted)
         self.assert_(self.smtp.closed)
 
+    def test_send_auth_unicode(self):
+        fromaddr = 'me@example.com'
+        toaddrs = ('you@example.com', 'him@example.com')
+        msgtext = 'Headers: headers\n\nbodybodybody\n-- \nsig\n'
+        self.mailer.username = u'f\u00f8\u00f8' # double o slash
+        self.mailer.password = u'\u00e9vil' # e acute
+        self.mailer.hostname = 'spamrelay'
+        self.mailer.port = 31337
+        self.mailer.send(fromaddr, toaddrs, msgtext)
+        self.assertEquals(self.smtp.username, 'f\xc3\xb8\xc3\xb8')
+        self.assertEquals(self.smtp.password, '\xc3\xa9vil')
+
+    def test_send_auth_nonascii(self):
+        fromaddr = 'me@example.com'
+        toaddrs = ('you@example.com', 'him@example.com')
+        msgtext = 'Headers: headers\n\nbodybodybody\n-- \nsig\n'
+        self.mailer.username = 'f\xc3\xb8\xc3\xb8' # double o slash
+        self.mailer.password = '\xc3\xa9vil' # e acute
+        self.mailer.hostname = 'spamrelay'
+        self.mailer.port = 31337
+        self.mailer.send(fromaddr, toaddrs, msgtext)
+        self.assertEquals(self.smtp.username, 'f\xc3\xb8\xc3\xb8')
+        self.assertEquals(self.smtp.password, '\xc3\xa9vil')
+
     def test_send_failQuit(self):
         self.mailer.smtp.fail_on_quit = True
         try:
@@ -162,6 +186,9 @@ def test_send_auth(self):
         # here, so pass.
         pass
 
+    test_send_auth_unicode = test_send_auth
+    test_send_auth_nonascii = test_send_auth
+
 def test_suite():
     suite = unittest.TestSuite()
     suite.addTest(unittest.makeSuite(TestSMTPMailer))