[issue8280] urllib2 passes fragment identifier to server
Senthil Kumaran orsent...@gmail.com added the comment: Fixed in revision 83818(py3k), 83819 (release31-maint) and 83820 (release27-maint). David, a couple of comments on your patch. - Request method was from urllib2, so the proper place of tests were test_urllib2. This already had Requests test so some additional tests were only required. - Also, now the fragments are removed and sent to the server, a proper response could be obtained, it can tested via 'network' tests in test_urllib2net. You might check the svn diffs to see the changes made. Thanks for the patch. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8280 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8280] urllib2 passes fragment identifier to server
Changes by Senthil Kumaran orsent...@gmail.com: -- resolution: accepted - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8280 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8280] urllib2 passes fragment identifier to server
David Stanek dsta...@dstanek.com added the comment: Added a patch to fix this behavior. -- keywords: +patch nosy: +dstanek Added file: http://bugs.python.org/file18358/fragment.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8280 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8280] urllib2 passes fragment identifier to server
David Stanek dsta...@dstanek.com added the comment: I have also uploaded my patch to http://codereview.appspot.com/1918042 so easier viewing. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8280 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8280] urllib2 passes fragment identifier to server
Changes by Senthil Kumaran orsent...@gmail.com: -- assignee: - orsenthil resolution: - accepted ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8280 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8280] urllib2 passes fragment identifier to server
New submission from INADA Naoki songofaca...@gmail.com: urllib2.urlopen(http://wave-robot-python-client.googlecode.com/svn/trunk/pydocs/index.html#module-wavelet;) Traceback (most recent call last): File stdin, line 1, in module File C:\usr\Python2.6\lib\urllib2.py, line 126, in urlopen return _opener.open(url, data, timeout) File C:\usr\Python2.6\lib\urllib2.py, line 398, in open response = meth(req, response) File C:\usr\Python2.6\lib\urllib2.py, line 511, in http_response 'http', request, response, code, msg, hdrs) File C:\usr\Python2.6\lib\urllib2.py, line 436, in error return self._call_chain(*args) File C:\usr\Python2.6\lib\urllib2.py, line 370, in _call_chain result = func(*args) File C:\usr\Python2.6\lib\urllib2.py, line 519, in http_error_default raise HTTPError(req.get_full_url(), code, msg, hdrs, fp) urllib2.HTTPError: HTTP Error 400: Bad Request This happens when redirected URL contains fragment. urllib2.urlopen(http://goo.gl/z1d5;) Traceback (most recent call last): File stdin, line 1, in module File C:\usr\Python2.6\lib\urllib2.py, line 126, in urlopen return _opener.open(url, data, timeout) File C:\usr\Python2.6\lib\urllib2.py, line 398, in open response = meth(req, response) File C:\usr\Python2.6\lib\urllib2.py, line 511, in http_response 'http', request, response, code, msg, hdrs) File C:\usr\Python2.6\lib\urllib2.py, line 430, in error result = self._call_chain(*args) File C:\usr\Python2.6\lib\urllib2.py, line 370, in _call_chain result = func(*args) File C:\usr\Python2.6\lib\urllib2.py, line 606, in http_error_302 return self.parent.open(new, timeout=req.timeout) File C:\usr\Python2.6\lib\urllib2.py, line 398, in open response = meth(req, response) File C:\usr\Python2.6\lib\urllib2.py, line 511, in http_response 'http', request, response, code, msg, hdrs) File C:\usr\Python2.6\lib\urllib2.py, line 436, in error return self._call_chain(*args) File C:\usr\Python2.6\lib\urllib2.py, line 370, in _call_chain result = func(*args) File C:\usr\Python2.6\lib\urllib2.py, line 519, in http_error_default raise HTTPError(req.get_full_url(), code, msg, hdrs, fp) urllib2.HTTPError: HTTP Error 400: Bad Request urllib2.Request.get_selector() should be: def get_selector(self): return self.__r_host.split('#')[0] -- messages: 102103 nosy: naoki severity: normal status: open title: urllib2 passes fragment identifier to server ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8280 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8280] urllib2 passes fragment identifier to server
Changes by INADA Naoki songofaca...@gmail.com: -- components: +Library (Lib) type: - behavior versions: +Python 2.5, Python 2.6, Python 2.7, Python 3.1, Python 3.2, Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8280 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8280] urllib2 passes fragment identifier to server
Changes by Brian Curtin cur...@acm.org: -- nosy: +orsenthil versions: -Python 2.5, Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8280 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8280] urllib2 passes fragment identifier to server
Changes by Éric Araujo mer...@netwok.org: -- nosy: +merwok ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8280 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com