Senthil Kumaran <sent...@uthcode.com> added the comment: On Sun, Jul 8, 2012 at 2:30 AM, Antoine Pitrou <rep...@bugs.python.org> wrote: > > Senthil, do you read python-dev? I think this change was prematurate from the > start (nevermind the fact that you didn't run the test suite before > committing).
I thought that the other legacy URLOpen was quoting it correct and then I wanted to see it can be made consistent. It did get me thinking that why it was different for so long. I realize that committing soon was a mistake. > For example, if you have an URL with a non-ASCII domain name such as > "http://وزارة-الأتصالات.مصر/", the domain name should IDNA-encoded, not > %-encoded like the rest. Agreed and understood. > Furthermore, some people are certainly already quoting their URLs to > workaround this issue, so "fixing" it will break their code by > double-escaping the URLs. You've got to be more careful. Oh. yes, the change may break an already quoted URL. I think, I shall revert this back. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue14826> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com