Message238497
I opened Issue 23703 about the funny doubled bar.com result. After backing out revision 901e4e52b20a, but with my patch here applied:
>>> urljoin('mailto:foo@', 'bar.com')
'mailto:bar.com'
which seems fairly sensible to me. A more awkward question is if this behaviour of my patch is reasonable:
>>> urljoin('mailto:person-foo/bar@example.net', 'bar.com')
'mailto:person-foo/bar.com'
Yet another option, similar to my “any_scheme=True” flag, might be to change from the “uses_relative” white-list to a “not_relative” black-list of URL schemes, so that urljoin() works for arbitrary schemes except for ones like “mailto:” that are in the hard-coded list. |
|
Date |
User |
Action |
Args |
2015-03-19 07:25:17 | martin.panter | set | recipients:
+ martin.panter, orsenthil, mher, berker.peksag, demian.brecht, madison.may |
2015-03-19 07:25:17 | martin.panter | set | messageid: <1426749917.66.0.97983354883.issue18828@psf.upfronthosting.co.za> |
2015-03-19 07:25:17 | martin.panter | link | issue18828 messages |
2015-03-19 07:25:17 | martin.panter | create | |
|