Message330888
| Author | devkral |
|---|---|
| Recipients | SilentGhost, devkral, orsenthil |
| Date | 2018-12-02.16:00:09 |
| SpamBayes Score | -1.0 |
| Marked as misclassified | Yes |
| Message-id | <1543766409.67.0.788709270274.issue35377@psf.upfronthosting.co.za> |
| In-reply-to |
| Content | |
|---|---|
first a correction; there is one protocol where it make sense:
file
I would change:
...
elif scheme:
...
to
...
elif scheme and scheme != "file":
...
Second: no it is not a correct behaviour. Urlunsplit is supposed to create a valid url from a tuple created by urlsplit. :/// is not a valid url (except for the file protocol).
Third: rstrip could be simplified to rstrip(":/") |
|
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2018-12-02 16:00:09 | devkral | set | recipients: + devkral, orsenthil, SilentGhost |
| 2018-12-02 16:00:09 | devkral | set | messageid: <1543766409.67.0.788709270274.issue35377@psf.upfronthosting.co.za> |
| 2018-12-02 16:00:09 | devkral | link | issue35377 messages |
| 2018-12-02 16:00:09 | devkral | create | |