Skip to content

GH-123599: Reject non-local authority in pathlib.Path.from_uri() on POSIX#123650

Closed
barneygale wants to merge 5 commits intopython:mainfrom
barneygale:gh-123599-reject-non-local
Closed

GH-123599: Reject non-local authority in pathlib.Path.from_uri() on POSIX#123650
barneygale wants to merge 5 commits intopython:mainfrom
barneygale:gh-123599-reject-non-local

Conversation

@barneygale
Copy link
Contributor

@barneygale barneygale commented Sep 3, 2024 •
edited by github-actions bot
Loading

Raise ValueError in pathlib.Path.from_uri() if the given file: URI specifies a non-empty, non-localhost authority, and we're running on a platform without support for UNC paths.

📚 Documentation preview 📚: https://cpython-previews--123650.org.readthedocs.build/
@barneygale
pythonGH-123599: Reject non-local authority in `pathlib.Path.from_uri…
0cee0c4 
…()` on POSIX

Raise `ValueError` in `pathlib.Path.from_uri()` if the given `file:` URI
specifies a non-empty, non-`localhost` authority, and we're running on a
platform without support for UNC paths.
@bedevere-app bedevere-app bot mentioned this pull request Sep 3, 2024
Closed
@barneygale barneygale requested a review from AA-Turner October 21, 2024 15:15
@barneygale
Copy link
Contributor Author

barneygale commented Oct 23, 2024

In fact, I might implement this by making urllib.request.pathname2url() and url2pathname() less awful. Marking as a draft...
@barneygale barneygale marked this pull request as draft October 23, 2024 01:54
@barneygale
Copy link
Contributor Author

barneygale commented Mar 19, 2025 •
edited
Loading

Closing in favour of #126844
@barneygale barneygale closed this Mar 19, 2025
@barneygale barneygale mentioned this pull request Mar 20, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs backport to 3.13 bugs and security fixes topic-pathlib

1 participant

@barneygale