This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author steven.daprano
Recipients SilentGhost, devkral, orsenthil, steven.daprano
Date 2018-12-03.11:39:41
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1543837181.85.0.788709270274.issue35377@psf.upfronthosting.co.za>
In-reply-to
Content
You haven't given a convincing reason that there is a problem that needs fixing, or if there is, that your patch is the right way to fix it.

You have already pointed out that there is at least one scheme where :/// is part of a valid URL: "file:///". Where there is one, there could be others, if not today, then in the future:

    spam:///eggs.cheese

There are well over 200 registered schemes:

https://www.iana.org/assignments/uri-schemes/uri-schemes.xhtml

I don't think it will be practical to put in a whitelist or a blacklist of schemes that are, or aren't, permitted to include :/// after the scheme. It is the caller's responsibility to use the correct scheme, without adding extra characters to the end.

I asked you to justify why this should be considered a bug in the library, rather than a bug in your code. I'm not an expert on URLs, but the functions look correct to me. If you can't justify why this is a bug in the library that needs fixing, rather than user-error, we should close this or change it to a request for better documentation.
History
Date User Action Args
2018-12-03 11:39:41steven.dapranosetrecipients: + steven.daprano, orsenthil, SilentGhost, devkral
2018-12-03 11:39:41steven.dapranosetmessageid: <1543837181.85.0.788709270274.issue35377@psf.upfronthosting.co.za>
2018-12-03 11:39:41steven.dapranolinkissue35377 messages
2018-12-03 11:39:41steven.dapranocreate