Fix URI, Request target terminology.
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 28/100
- Loại issue
- Tài liệu
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- rust
- Lĩnh vực
- backend-api-design, documentation
Hướng nghiên cứu
Bắt đầu với tài liệu về struct Uri hiện tại của crate và cách xử lý request-target, sau đó xem xét các issue liên quan được liệt kê trong báo cáo. Công việc được xem là hoàn tất khi thuật ngữ mô tả nhất quán các dạng request-target và việc thay đổi tên công khai có một quyết định rõ ràng về khả năng tương thích trước khi triển khai.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
As this crate stands for correctness, it is imperative to fix misleading terminology that is used through out the request-target handling documentation.
- Use
HttpRequestTargetinstead ofUri, as it really handles only request target forms, not identity scheme in any way. This may be difficult, noting it is a breaking change. Can alias and deprecate though. - In documentation of current
Uristruct, There is extensive use ofabsolute-uri/relative-uridichotomy to show that, forabsolute-uriit hasport,scheme, etc, and forrelative-uriit won't. This is grossly misleading. As this crate is not handling identifiers in the first place, there is no such thing asabsolute-uri/relative-uridichotomy. It is instead dichotomy ofabsolute-form/origin-formof request target. i.e, request-target inabsolute-formencodes port, scheme etc, and that inorigin-formdoesn't, as they have to be runtime-resolved. That's also the reason the mis-termedrelative-uris starts with a/, as they are not really identifiers, relative/absolute, butorigin-formresource targets.
The second point of fixing documentation may not be breaking change, and possible i hope.
We can see many misunderstandings due to this misleading. for example
- #127, As These are not identifiers, but representation of request-target values, there is no uri, and thus no fragments in first place.
- #465 Same mis-issue of using request-target representation to represent identifiers
- #469 They even mention other uri-schemes in issue discription, and also quotes uri-standard. Same mis-issue.
- #396 ,
urlcrate is for identifiers, where as this one only deals with request-targets. - #379 same issue. This crate doesn't handles uris in first place. only request targets.
- #342
- #311
- #323
- #421
- #306
- #176
- #110
so on, so forth.
For amount of confusion, ad misleading it creates, It may be helpful to fix at least terminology in docs, and fix Uri name for struct before 1.0
- Ngôn ngữ chính
- Rust
- Star
- 1.4k
- Fork
- 378
- Merge trung bình
- 1 ngày 21 giờ
- Pull request đã merge (30 ngày)
- 5
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.
Issue khác của hyperium/http
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 62/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 58/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 62/100
Tất cả issue của hyperium/http
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
TheLarkInn/aipm#2413 ·
-
documentation
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
alexgorbatchev/simple-ptt#15 ·
-
tooling
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
todo:ticket
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
taikoxyz/taiko-mono#22168 · 1 bình luận ·