optimize code for proxy handling of `urllib.request`
Đang mở
Chưa có ai nhận issue này.
performance
stdlib
type-feature
- Ngôn ngữ chính
- Python
- Star
- 77.2k
- Fork
- 35.9k
- Chỉ số merge pull request
- Chỉ số pull request đang chờ
Mô tả
Feature or enhancement
Proposal:
- the second loop of line starts from 1845 is duplicated from the first loop.
https://github.com/python/cpython/blob/2041a95e68ebf6d13f867e214ada28affa830669/Lib/urllib/request.py#L1817-L1853 - the registry access should be separated to a standalone function for cache, to be used by code like https://github.com/aio-libs/aiohttp/blob/e79b2d5df70a2644e81925cc49558962af91848d/aiohttp/client.py#L609-L617
https://github.com/python/cpython/blob/2041a95e68ebf6d13f867e214ada28affa830669/Lib/urllib/request.py#L2071-L2104 - the proxy setting from registry should be ignored when if proxies from environment exists? like the
proxy_bypass()
Should open a PR?
Has this already been discussed elsewhere?
This is a minor feature, which does not need previous discussion elsewhere
Links to previous discussion of this feature:
No response
Linked PRs
- gh-127767
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.
Hướng nghiên cứu
Bắt đầu trong Lib/urllib/request.py, quanh các dòng 1817-1853 và 2071-2104, sau đó so sánh cách xử lý proxy của aiohttp được liên kết. Làm rõ liệu mục tiêu có bao gồm việc loại bỏ trùng lặp giữa các vòng lặp, tách quyền truy cập vào registry đã được lưu vào bộ nhớ đệm và ưu tiên các proxy môi trường hay không; công việc được xem là hoàn tất khi hành vi proxy đã thống nhất được triển khai và được bao phủ bởi các bài kiểm thử urllib.request phù hợp.
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
- Tính năng
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 25/100