Ankush Sharma has proposed merging 
lp:~black-perl/mailman/handling-special-chars-in-email into lp:mailman.

Requested reviews:
  Mailman Coders (mailman-coders)

For more details, see:
https://code.launchpad.net/~black-perl/mailman/handling-special-chars-in-email/+merge/252867

In case the mailman-client percent encodes any special character appeared in 
the list_id's or fqdn_listname's, the REST server should be able to decode it 
back. Added decoding support for it.
As discussed here: 
http://www.mail-archive.com/mailman-developers%40python.org/msg15317.html
Now mailman can also use emails with special characters [1] which earlier 
caused postorius to crash as discussed in the linked bug report.

[1] : http://oi58.tinypic.com/33u689i.jpg
-- 
Your team Mailman Coders is requested to review the proposed merge of 
lp:~black-perl/mailman/handling-special-chars-in-email into lp:mailman.
=== modified file 'src/mailman/rest/root.py'
--- src/mailman/rest/root.py	2015-02-13 08:13:06 +0000
+++ src/mailman/rest/root.py	2015-03-13 09:28:31 +0000
@@ -23,6 +23,7 @@
 
 
 import falcon
+import urllib.parse
 
 from base64 import b64decode
 from mailman.config import config
@@ -192,7 +193,9 @@
         else:
             # list-id is preferred, but for backward compatibility,
             # fqdn_listname is also accepted.
-            list_identifier = segments.pop(0)
+            enc_list_identifier = segments.pop(0)
+            # decoding the percent encoded list_identifier
+            list_identifier = urllib.parse.unquote(enc_list_identifier)
             return AList(list_identifier), segments
 
     @child()

_______________________________________________
Mailman-coders mailing list
[email protected]
https://mail.python.org/mailman/listinfo/mailman-coders

Reply via email to