urllib.request.url2pathname() mishandles non-empty authority sections on POSIX #126838

barneygale · 2024-11-14T20:35:34Z

Bug report

Bug description:

URLs supplied to urllib.request.url2pathname() may include a non-empty authority (netloc), such as a hostname.

On Windows, these are correctly decoded (mostly):

>>> url2pathname('//localhost/c:/foo.txt')
'C:\\foo.txt'  # OK
>>> url2pathname('//server/share')
'\\\\server\\share'  # OK

But on POSIX, any non-empty authority except localhost is preserved in the path:

>>> url2pathname('//myhostname/etc/hosts')
'//myhostname/etc/hosts'  # expected '/etc/hosts'
>>> url2pathname('//server/share')
'//server/share'  # expected URLError ('server' is not local)

Per RFC 8089 this isn't correct.

CPython versions tested on:

CPython main branch

Operating systems tested on:

Linux

Linked PRs

GH-123599: url2pathname(): handle authority section in file URL #126844

The text was updated successfully, but these errors were encountered:

… on POSIX Adjust `urllib.request.url2pathname()` to parse the URL authority and path with `urlsplit()` on POSIX. If the authority is empty or resolves to the current host, it is ignored and the URL path is used as the pathname. If not, we raise `URLError`.

barneygale · 2024-11-27T16:51:37Z

Closing - dupe of #123599

barneygale added type-bug An unexpected behavior, bug, or error stdlib Python modules in the Lib dir 3.12 only security fixes 3.13 bugs and security fixes 3.14 bugs and security fixes labels Nov 14, 2024

bedevere-app bot mentioned this issue Nov 14, 2024

GH-123599: url2pathname(): handle authority section in file URL #126844

Merged

barneygale added a commit to barneygale/cpython that referenced this issue Nov 19, 2024

Merge branch 'main' into pythongh-126838

96fdb01

barneygale added a commit to barneygale/cpython that referenced this issue Nov 19, 2024

Merge branch 'main' into pythongh-126838

52010ca

barneygale closed this as not planned Won't fix, can't repro, duplicate, stale Nov 27, 2024

barneygale added a commit to barneygale/cpython that referenced this issue Mar 19, 2025

Merge branch 'main' into pythongh-126838

f64ca0d

barneygale added a commit to barneygale/cpython that referenced this issue Mar 19, 2025

Merge branch 'main' into pythongh-126838

69807b4

barneygale added a commit to barneygale/cpython that referenced this issue Mar 20, 2025

Merge branch 'main' into pythongh-126838

b4fa510

barneygale added a commit to barneygale/cpython that referenced this issue Apr 10, 2025

Merge branch 'main' into pythongh-126838

eecff9b

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

urllib.request.url2pathname() mishandles non-empty authority sections on POSIX #126838

urllib.request.url2pathname() mishandles non-empty authority sections on POSIX #126838

Uh oh!

Uh oh!

urllib.request.url2pathname() mishandles non-empty authority sections on POSIX #126838

urllib.request.url2pathname() mishandles non-empty authority sections on POSIX #126838

Comments

Uh oh!

Bug report

Bug description:

CPython versions tested on:

Operating systems tested on:

Linked PRs

Uh oh!