urljoin works incorrectly for two path-relative URLs involving . and ..
@serhiy-storchaka đang làm issue này rồi.
Từ ngày 11/11/2024.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Bug report
urllib.parse.urljoin is usually used to join a normalized absolute URL with a relative URL, and it generally works for that purpose. But if it’s used to join two path-relative URLs, it produces incorrect results in many cases when . or .. is involved.
>>> from urllib.parse import urljoin
>>> urljoin('a', 'b') # ok
'b'
>>> urljoin('a/', 'b') # ok
'a/b'
>>> urljoin('a', '.') # expected . or ./
'/'
>>> urljoin('a', '..') # expected .. or ../
'/'
>>> urljoin('..', 'b') # expected ../b
'b'
>>> urljoin('../a', 'b') # expected ../b
'b'
>>> urljoin('a', '../b') # expected ../b
'b'
>>> urljoin('../a', '../b') # expected ../../b
'b'
>>> urljoin('a/..', 'b') # expected b
'a/b'
There are also some problems when the base is a non-normalized absolute URL:
>>> urljoin('http://host/a/..', 'b') # expected http://host/b
'http://host/a/b'
Your environment
- CPython versions tested on: 3.10.5, 3.11.0rc1
- Operating system and architecture: NixOS 21.11 amd64
- Ngôn ngữ chính
- Python
- Star
- 77.2k
- Fork
- 36k
- Merge trung bình
- 1 ngày 9 giờ
- Pull request đã merge (30 ngày)
- 558
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của python/cpython
-
docs pending
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
stdlib type-feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
stdlib type-feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
build type-bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
-
stdlib topic-email type-feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Tất cả issue của python/cpython
Issue tương tự
-
area/auth bug comp/agent P3 platform/discord type/security
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
NousResearch/hermes-agent#117848 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
bancolombia/sentinel#23 ·
-
test md Đang mởCI
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
-
integration:quickjs org:external priority:backlog topic:code-interpreter topic:middleware type:feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
langchain-ai/deepagents#6450 ·
-
bug client
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100