➜
This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.
Created on2015-03-19 07:09 bymartin.panter, last changed2022-04-11 14:58 byadmin. This issue is nowclosed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| issue23703.patch | demian.brecht,2015-03-19 15:05 | review | ||
| issue23703_1.patch | demian.brecht,2015-03-20 00:13 | review | ||
| Messages (7) | |||
|---|---|---|---|
| msg238493 -(view) | Author: Martin Panter (martin.panter)* | Date: 2015-03-19 07:09 | |
This is a regression caused by revision901e4e52b20a. Before (e.g. Python 3.4):>>> urljoin('a', 'b')'b'After:>>> urljoin('a', 'b')'b/b'This was identified in <https://bugs.python.org/issue18828#msg238363> but is actually caused by the revision committed forIssue 22278. | |||
| msg238531 -(view) | Author: Demian Brecht (demian.brecht)* | Date: 2015-03-19 15:05 | |
Nice work tracking down the commit, thanks for that. I've attached a fix. | |||
| msg238566 -(view) | Author: Martin Panter (martin.panter)* | Date: 2015-03-19 21:57 | |
Patch looks good enough.BTW I was thinking of something like this, which is a bit simpler, but I never tested it:segments[1:-1] = filter(None, segments[1:-1]) | |||
| msg238590 -(view) | Author: Demian Brecht (demian.brecht)* | Date: 2015-03-20 00:13 | |
Yep, that's a lot cleaner and tests pass. I've updated the patch with that line. Thanks. | |||
| msg241082 -(view) | Author: Martin Panter (martin.panter)* | Date: 2015-04-15 05:41 | |
Any chance this regression can be fixed before the next release? | |||
| msg241177 -(view) | Author: Roundup Robot (python-dev) | Date: 2015-04-15 23:31 | |
New changesetfc0e79387a3a by Berker Peksag in branch 'default':Issue#23703: Fix a regression in urljoin() introduced in901e4e52b20a.https://hg.python.org/cpython/rev/fc0e79387a3a | |||
| msg241178 -(view) | Author: Berker Peksag (berker.peksag)* | Date: 2015-04-15 23:34 | |
Thanks! | |||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:14 | admin | set | github: 67891 |
| 2015-04-15 23:34:22 | berker.peksag | set | status: open -> closed resolution: fixed messages: +msg241178 stage: patch review -> resolved |
| 2015-04-15 23:31:37 | python-dev | set | nosy: +python-dev messages: +msg241177 |
| 2015-04-15 05:41:01 | martin.panter | set | messages: +msg241082 |
| 2015-03-24 14:16:13 | berker.peksag | set | nosy: +berker.peksag |
| 2015-03-20 00:13:42 | demian.brecht | set | files: +issue23703_1.patch messages: +msg238590 |
| 2015-03-19 21:57:04 | martin.panter | set | messages: +msg238566 |
| 2015-03-19 15:05:18 | demian.brecht | set | files: +issue23703.patch keywords: +patch messages: +msg238531 stage: patch review |
| 2015-03-19 08:15:58 | berker.peksag | set | nosy: +orsenthil |
| 2015-03-19 07:09:16 | martin.panter | create | |