python-hyper / python-hyper/rfc3986
Resolution result is not consistent with RFC 3986 when the base has "rootless" path without authority
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 189
- Fork
- 38
- Merge trung bình
- 13 giờ 36 phút
- Pull request đã merge (30 ngày)
- 1
Mô tả
remove_dot_segments defined in RFC 3986 (section 5.2.4) sometimes add a leading slash when the base path is "rootless".
STEP OUTPUT BUFFER INPUT BUFFER
1 : foo/../baz
2E: foo /../baz
2C: /baz
2E: /baz
So, resolving ../baz against scheme:foo/bar should result in scheme:/baz.
However, output of this library differs from that.
$ python
Python 3.9.9 (main, Jan 10 2022, 18:52:39)
[GCC 11.2.1 20211127] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from rfc3986 import uri_reference
>>> b = uri_reference('scheme:foo/bar')
>>> r = uri_reference('../baz')
>>> t = r.resolve_with(b)
>>> t
URIReference(scheme='scheme', authority=None, path='baz', query=None, fragment=None)
>>> t.unsplit()
'scheme:baz'
>>>
I think there should be special handling such as "when .. segment appears but output stack is empty, set prepend_slash flag" or something like that.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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.
Hướng nghiên cứu
Bắt đầu với các quy tắc trong mục 5.2.4 của RFC 3986 và các entry point uri_reference, resolve_with và remove_dot_segments. Tái hiện ví dụ phân giải ../baz với scheme:foo/bar; hoàn thành khi kết quả được ghép lại thành scheme:/baz trong khi vẫn giữ nguyên hành vi phân giải hiện có.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- networking
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 48/100