Skip to content

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

Closed as not planned
Closed as not planned
urllib.request.url2pathname() mishandles non-empty authority sections on POSIX#126838
Labels
3.12only security fixes3.13only security fixes3.14bugs and security fixesstdlibStandard Library Python modules in the Lib/ directorytype-bugAn unexpected behavior, bug, or error

Description

@barneygale
barneygale
opened on Nov 14, 2024

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

Activity

  1. barneygale
    added
    type-bugAn unexpected behavior, bug, or error
    stdlibStandard Library Python modules in the Lib/ directory
    3.12only security fixes
    3.13only security fixes
    3.14bugs and security fixes
    on Nov 14, 2024
  2. barneygale
    added 3 commits that reference this issue on Nov 14, 2024

    pythonGH-126838: `url2pathname()`: handle non-empty authority section…

    50231c5

    Merge branch 'main' into pythongh-126838

    96fdb01

    Merge branch 'main' into pythongh-126838

    52010ca

  3. barneygale commented on Nov 27, 2024

    @barneygale
    barneygale
    on Nov 27, 2024
    ContributorAuthor

    Closing - dupe of #123599

  4. barneygale
    closed this as not plannedon Nov 27, 2024
  5. barneygale
    added 3 commits that reference this issue on Mar 19, 2025

    Merge branch 'main' into pythongh-126838

    f64ca0d

    Merge branch 'main' into pythongh-126838

    69807b4

    Merge branch 'main' into pythongh-126838

    b4fa510
  6. barneygale
    added a commit that references this issue on Apr 10, 2025

    Merge branch 'main' into pythongh-126838

    eecff9b
  7. agoscinski
    added 8 commits that reference this issue on Feb 28, 2026

    Replace `urllib.request.url2pathname` with local shim for Python 3.14

    03a0f63

    Replace `urllib.request.url2pathname` with local shim for Python 3.14

    b4ca165

    Replace `urllib.request.url2pathname` with local shim for Python 3.14

    6405e1e

    Replace `urllib.request.url2pathname` with local shim for Python 3.14 (…

    e007df4

    Replace `urllib.request.url2pathname` with local shim for Python 3.14 (…

    a062fb4

    Replace `urllib.request.url2pathname` with local shim for Python 3.14 (…

    f1c8337

    Replace `urllib.request.url2pathname` with local shim for Python 3.14 (…

    3e6e2a3

    Replace `urllib.request.url2pathname` with local shim for Python 3.14 (…

    1607da4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    3.12only security fixes3.13only security fixes3.14bugs and security fixesstdlibStandard Library Python modules in the Lib/ directorytype-bugAn unexpected behavior, bug, or error

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions