[Merge] lp:~black-perl/mailman.client/handling-special-chars-in-email into lp:mailman.client

Ankush Sharma ankprashar at gmail.com
Fri Mar 13 15:17:27 CET 2015


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

Requested reviews:
  Mailman Coders (mailman-coders)
Related bugs:
  Bug #1429366 in GNU Mailman REST Client: "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.client/+bug/1429366

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

As discussed in the bug report, use of emails with special character set is valid as per email RFCs but postorius crashes on using them giving 404 or KeyError always.
As discussed on the mailing list a possible solution would be to percent encode these special characters when they appear in the list_id or fqdn_listname before sending a request to the REST server and decoding on the other end.
Added utils.py which has to functions `encode` and `encode_url` to faciliate encoding of list_id's in URLs when required.
The use of special characters in list_ids is working properly http://oi58.tinypic.com/33u689i.jpg.
-- 
Your team Mailman Coders is requested to review the proposed merge of lp:~black-perl/mailman.client/handling-special-chars-in-email into lp:mailman.client.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 5473 bytes
Desc: not available
URL: <http://mail.python.org/pipermail/mailman-coders/attachments/20150313/35696a68/attachment.diff>


More information about the Mailman-coders mailing list