Ankush Sharma has proposed merging
lp:~black-perl/mailman/handling-special-chars-in-email into lp:mailman.
Requested reviews:
Mailman Coders (mailman-coders)
Related bugs:
Bug #1429366 in GNU Mailman: "Anatomy of list ids does not keep with that of
urls causes some REST end points to return 404 always"
https://bugs.launchpad.net/mailman/+bug/1429366
For more details, see:
https://code.launchpad.net/~black-perl/mailman/handling-special-chars-in-email/+merge/252865
In case the mailman-client percent encodes any appeared special characters in
the list_identifier or fqdn_listname the mailman's REST server should be able
to decode it back. Added support for this decoding operation.As discussed here
: http://www.mail-archive.com/mailman-developers%40python.org/msg15317.html
--
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:12:32 +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(list_identifier)
return AList(list_identifier), segments
@child()
_______________________________________________
Mailman-coders mailing list
[email protected]
https://mail.python.org/mailman/listinfo/mailman-coders
