[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-12-16 Thread Senthil Kumaran
Senthil Kumaran added the comment: Fixed in r87329, r87330 and r87331. Thanks for the patch Wes, that was proper. -- resolution: accepted -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-11-20 Thread Wes Chow
Wes Chow added the comment: Patch against 2.7 (urlparse-9721-2.7.patch). -- Added file: http://bugs.python.org/file19707/urlparse-9721-2.7.patch ___ Python tracker ___ __

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-11-20 Thread Wes Chow
Wes Chow added the comment: New patch (urlparse-9721-3.2.patch) against 3.2 which fixes the erroneous test. -- Added file: http://bugs.python.org/file19706/urlparse-9721-3.2.patch ___ Python tracker ___

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-11-20 Thread R. David Murray
Changes by R. David Murray : -- versions: +Python 2.7, Python 3.2 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-11-20 Thread R. David Murray
Changes by R. David Murray : -- stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-11-20 Thread Wes Chow
Wes Chow added the comment: Here's a patch for 3.2 which fixes this problem I believe. There does exist a test case that should have produced an error, except that the last path segment in RFC3986_BASE is only one character long. Had it been more than one character long, RFC3986 checks would

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-09-06 Thread Senthil Kumaran
Senthil Kumaran added the comment: Simply applying the patch would break the testsuite of urlparse ( test_RFC3986 case of urljoin for ';' starting joinurls). The expected behavior should be trim off all receding chars until a '/' and then join the semi-colon starting url. 2.5 behavior is the

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-09-03 Thread Éric Araujo
Éric Araujo added the comment: The versions getting bug fixes are 2.7, 3.1 (stable versions) and 3.2 (active). 2.6 is in security mode now. Can someone write a test (as a standalone script or a diff against Lib/test/test_urlparse.py) and tell if 3.1 and 3.2 have the bug too? More importantly

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-08-31 Thread R. David Murray
Changes by R. David Murray : -- nosy: +orsenthil ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyt

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-08-31 Thread Michał Powaga
Michał Powaga added the comment: The problem was be here: path = path[:-1] # This is not needed and cuts last character return urlunparse((scheme, netloc, path, params, query, fragment)) I sent a patch. PS. Sorry if I`m doing something wrong but it is my first patch and acti

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-08-30 Thread Bastian Kleineidam
Bastian Kleineidam added the comment: Update: the python2.5 behaviour is the expected and what I think the correct output. -- versions: -Python 2.5 ___ Python tracker ___ _

[issue9721] urlparse.urljoin() cuts off last base character with semicolon at url start

2010-08-30 Thread Bastian Kleineidam
New submission from Bastian Kleineidam : The urljoin() implementation cuts off the last base URL character if the URL to join starts with a semicolon. Expected output is no cut off characters. $ python2.6 Python 2.6.6 (r266:84292, Aug 29 2010, 12:36:23) [GCC 4.4.5 20100824 (prerelease)] on linu