[issue2830] Copy cgi.escape() to html
Pablo Mouzo added the comment: I'm attaching a patch against py3k trunk that moves the function to the html module and fixes the documentation as Brett asked for. It also changes all the occurrences of cgi.escape I found for html.escape . -- keywords: +patch nosy: +pablomouzo Added file: http://bugs.python.org/file18612/issue2830.diff ___ Python tracker <http://bugs.python.org/issue2830> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2830] Copy cgi.escape() to html
Pablo Mouzo added the comment: Thanks Georg for the review, I'm attaching a new patch with those problems fixed. The new patch escapes ' when the quote parameter is true, and / always. -- Added file: http://bugs.python.org/file18619/issue2830.diff ___ Python tracker <http://bugs.python.org/issue2830> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2830] Copy cgi.escape() to html
Changes by Pablo Mouzo : Removed file: http://bugs.python.org/file18612/issue2830.diff ___ Python tracker <http://bugs.python.org/issue2830> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2830] Copy cgi.escape() to html
Changes by Pablo Mouzo : Removed file: http://bugs.python.org/file18619/issue2830.diff ___ Python tracker <http://bugs.python.org/issue2830> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2830] Copy cgi.escape() to html
Pablo Mouzo added the comment: Sorry about that, I have no idea how I managed to generate that diff. I'm attaching the correct patch. About the slash, there's a link in #9061 that recommends to escape the slash too because it's used to close tags in HTML. Is there any chance that escaping it could cause problems? -- Added file: http://bugs.python.org/file18636/issue2830.diff ___ Python tracker <http://bugs.python.org/issue2830> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2830] Copy cgi.escape() to html
Changes by Pablo Mouzo : Removed file: http://bugs.python.org/file18636/issue2830.diff ___ Python tracker <http://bugs.python.org/issue2830> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2830] Copy cgi.escape() to html
Pablo Mouzo added the comment: I'm attaching a new patch without escaping the slash. -- Added file: http://bugs.python.org/file18667/issue2830.diff ___ Python tracker <http://bugs.python.org/issue2830> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2830] Copy cgi.escape() to html
Pablo Mouzo added the comment: I'm attaching a new patch with the documentation updated. I agree with Georg that it'd be better to escape everything by default. Are there any good reasons not to? -- Added file: http://bugs.python.org/file18677/issue2830.diff ___ Python tracker <http://bugs.python.org/issue2830> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2830] Copy cgi.escape() to html
Changes by Pablo Mouzo : Removed file: http://bugs.python.org/file18667/issue2830.diff ___ Python tracker <http://bugs.python.org/issue2830> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11664] Add patch method to unittest.TestCase
Pablo Mouzo added the comment: I'm attaching a draft patch for patch. Éric, is this patch implementing patch as you expected? This patch is not finished because there are many cases where patch can leave patched objects if it fails to unpatch. -- keywords: +patch nosy: +pablomouzo Added file: http://bugs.python.org/file21412/patch.diff ___ Python tracker <http://bugs.python.org/issue11664> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue7723] sqlite only accept buffer() for BLOB objects (input/output)
Pablo Mouzo added the comment: I'm attaching a correction to sqlite3 documentation, removing all 'buffer' references and setting bytes as BLOB equivalent. -- keywords: +patch nosy: +pablomouzo Added file: http://bugs.python.org/file15977/issue7723-py3k-docs.diff ___ Python tracker <http://bugs.python.org/issue7723> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6045] Fix dbm interfaces
Changes by Pablo Mouzo : -- nosy: +pablomouzo ___ Python tracker <http://bugs.python.org/issue6045> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue7911] unittest.TestCase.longMessage should default to True in Python 3.2
Pablo Mouzo added the comment: The patch changes the default to True, and updates the tests and the docs to reflect this. -- keywords: +patch nosy: +pablomouzo Added file: http://bugs.python.org/file16377/issue7834.diff ___ Python tracker <http://bugs.python.org/issue7911> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue7300] Unicode arguments in str.format()
Changes by Pablo Mouzo : -- nosy: +pablomouzo ___ Python tracker <http://bugs.python.org/issue7300> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] ftplib rest support for storbinary
New submission from Pablo Mouzo : FPT class doesn't support the rest parameter in storbinary method. -- components: None messages: 92287 nosy: pablomouzo severity: normal status: open title: ftplib rest support for storbinary type: feature request versions: Python 2.7 ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] ftplib rest support for storbinary
Pablo Mouzo added the comment: I attached a patch. -- keywords: +patch Added file: http://bugs.python.org/file14841/issue6845.patch ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] ftplib rest support for storbinary
Changes by Pablo Mouzo : -- nosy: +facundobatista ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] ftplib rest support for storbinary
Changes by Pablo Mouzo : Removed file: http://bugs.python.org/file14841/issue6845.patch ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] Restart support in binary upload for ftplib
Pablo Mouzo added the comment: I'm changing the title to something clearer (I hope). The patch I submitted adds the retr optional parameter to the storbinary method. -- title: ftplib rest support for storbinary -> Restart support in binary upload for ftplib Added file: http://bugs.python.org/file14876/issue6845.patch ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] Restart support in binary upload for ftplib
Pablo Mouzo added the comment: I attached some tests. -- Added file: http://bugs.python.org/file15027/issue6845.diff ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] Restart support in binary upload for ftplib
Changes by Pablo Mouzo : Removed file: http://bugs.python.org/file14876/issue6845.patch ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] Restart support in binary upload for ftplib
Changes by Pablo Mouzo : -- nosy: +gregory.p.smith ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] Restart support in binary upload for ftplib
Pablo Mouzo added the comment: I think that a non-digit REST argument is an error. But I've tested some ftp servers and they don't return an error code, they just set REST to 0 and return some OK code. Finally, ftplib doesn't check that, so if someone accidentally passes a non-digit argument it could be a difficult bug to spot. -- ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] Restart support in binary upload for ftplib
Pablo Mouzo added the comment: Here is a new patch. It works with both ints and strings. I'm working on a patch for py3k. -- Added file: http://bugs.python.org/file15302/issue6845-trunk.diff ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] Restart support in binary upload for ftplib
Changes by Pablo Mouzo : Removed file: http://bugs.python.org/file15027/issue6845.diff ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] Restart support in binary upload for ftplib
Changes by Pablo Mouzo : Removed file: http://bugs.python.org/file15302/issue6845-trunk.diff ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] Restart support in binary upload for ftplib
Pablo Mouzo added the comment: Thanks Antoine, I'm attaching both patches. Now the test isn't failing. -- Added file: http://bugs.python.org/file15401/issue6845-trunk.diff ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6845] Restart support in binary upload for ftplib
Changes by Pablo Mouzo : Added file: http://bugs.python.org/file15402/issue6845-py3k.diff ___ Python tracker <http://bugs.python.org/issue6845> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue7484] smtplib: verify breaks with Postfix servers
Pablo Mouzo added the comment: This patch solves the problem with the VRFY command, but I'm still wondering if this happens with other commands too. -- keywords: +patch nosy: +pablomouzo Added file: http://bugs.python.org/file15674/issue7484-trunk.diff ___ Python tracker <http://bugs.python.org/issue7484> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue7484] smtplib: verify breaks with Postfix servers
Changes by Pablo Mouzo : Added file: http://bugs.python.org/file15675/issue7484-py3k.diff ___ Python tracker <http://bugs.python.org/issue7484> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue7540] urllib2 request does not update content length after new add_data
Pablo Mouzo added the comment: The problem here is that the headers are not updated if they already exists. The solution is quite simple but breaks the tests because it "clobbers the existing headers". You can do this: ... req.add_data(some_data) req.add_unredirected_header('Content-Length', len(some_data)) urllib2.urlopen(req) ... But is risky because all the other headers are still outdated. Is there any reason why you need to reuse the request object? -- keywords: +patch nosy: +pablomouzo Added file: http://bugs.python.org/file15680/bad_fix.diff ___ Python tracker <http://bugs.python.org/issue7540> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com