google-auth: connection leaks in urllib3 transport and metadata helper during mTLS updates
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 72/100
Hướng nghiên cứu
Đọc google/auth/transport/urllib3.py và google/auth/compute_engine/_metadata.py, bắt đầu với configure_mtls_channel và đường dẫn mount MdsMtlsAdapter cho từng request. Xác nhận rằng PoolManager cũ được dọn dẹp và metadata adapter được tái sử dụng hoặc đóng thay vì tích lũy socket, sau đó thêm hoặc chạy các kiểm thử hồi quy tập trung cho cả hai đường dẫn.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
When mTLS is configured or dynamically updated, the SDK creates and mounts new HTTP adapters/transports but leaves the old ones hanging. While PR #17689 addressed this for the requests transport, the leak still exists in two other places:
-
urllib3 transport (
google/auth/transport/urllib3.py)
Inconfigure_mtls_channel,self.http = new_httpreplaces the old PoolManager. We need to call.clear()on the oldPoolManagerinstance before overwriting it so that the active connection pool is cleaned up. -
Compute Engine metadata (
google/auth/compute_engine/_metadata.py)
If mTLS is enabled,_metadata.pyinstantiates and mounts a newMdsMtlsAdapteron the session for every single request. Overwriting the mounted adapter without closing the old one leaks sockets. We should either cache and reuse the adapter or close the old one before mounting.
- Ngôn ngữ chính
- Python
- Star
- 5.4k
- Fork
- 1.8k
- Merge trung bình
- 2 ngày 12 giờ
- Pull request đã merge (30 ngày)
- 143
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc 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 googleapis/google-cloud-python
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
googleapis/google-cloud-python#18532 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 92/100
googleapis/google-cloud-python#18467 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
auth priority: p2 type: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
googleapis/google-cloud-python#18428 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
priority: p2 type: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
googleapis/google-cloud-python#18375 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
ci: enable --strict flag in presubmit twine_checkCó thể đã có người làm @ohmayr đã nhận 2 ngày trước. Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 76/100
googleapis/google-cloud-python#18339 · 1 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của googleapis/google-cloud-python
Issue tương tự
-
good first issue hacktoberfest
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
RogueAlg0/taken#387 · 4 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
tool-calling
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
vllm-project/vllm#59838 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
raullenchai/Rapid-MLX#4042 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
documentation
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
transitmatters/mbta-slow-zone-bot#70 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
open-webui/open-webui#31871 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày