------------------------------------------------------------
revno: 1225
committer: Mark Sapiro <msap...@value.net>
branch nick: 2.2
timestamp: Thu 2012-09-06 16:57:00 -0700
message:
  Fixed a bug where non-ascii characters in the real name in a subscription
  request could throw a UnicodeEncodeError upon subscription approval and
  perhaps in other situations too.  (LP: 1047100)
modified:
  Mailman/Logging/Syslog.py
  NEWS


--
lp:mailman/2.2
https://code.launchpad.net/~mailman-coders/mailman/2.2

Your team Mailman Checkins is subscribed to branch lp:mailman/2.2.
To unsubscribe from this branch go to 
https://code.launchpad.net/~mailman-coders/mailman/2.2/+edit-subscription
=== modified file 'Mailman/Logging/Syslog.py'
--- Mailman/Logging/Syslog.py	2005-12-30 18:50:08 +0000
+++ Mailman/Logging/Syslog.py	2012-09-06 23:57:00 +0000
@@ -62,6 +62,10 @@
             logf.write(msg + '\n')
         except UnicodeError:
             # Python 2.4 may fail to write 8bit (non-ascii) characters
+            # Also, if msg is unicode with non-ascii, quopri.encodestring()
+            # will throw UnicodeEncodeError, so avoid that.
+            if isinstance(msg, unicode):
+                msg = msg.encode('iso-8859-1', 'replace')
             logf.write(quopri.encodestring(msg) + '\n')
 
     # For the ultimate in convenience

=== modified file 'NEWS'
--- NEWS	2012-08-23 04:02:29 +0000
+++ NEWS	2012-09-06 23:57:00 +0000
@@ -53,6 +53,10 @@
 
   Bug Fixes and other patches
 
+    - Fixed a bug where non-ascii characters in the real name in a subscription
+      request could throw a UnicodeEncodeError upon subscription approval and
+      perhaps in other situations too.  (LP: 1047100)
+
     - The query fragments send_unsub_notifications_to_list_owner and
       send_unsub_ack_to_this_batch will now assume default values if not set
       in mass unsubscribe URLs.  (LP: #1032378)

_______________________________________________
Mailman-checkins mailing list
Mailman-checkins@python.org
Unsubscribe: 
http://mail.python.org/mailman/options/mailman-checkins/archive%40jab.org

Reply via email to