John Edmonds <pocketcooki...@gmail.com> added the comment: Thanks for reviewing the patch. I don't believe I received an email for the review. I think I have addressed your comment about the usage of str(body) by removing the call to str() and changing the tests to use byte literals.
As for the content-length changing in the tests, that is because encode_multipart uses '\r\n' to end lines while the previous code only ended lines in '\n'. ---------- Added file: http://bugs.python.org/file22478/patch.diff _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue12169> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com