Deadlock in mutually exclusive account_id and zone_id resolution raises erroneous ValueError
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 45/100
Hướng nghiên cứu
Tìm trong codebase _get_account_id_path_param, _get_zone_id_path_param và mẫu account_id/zone_id loại trừ lẫn nhau; bắt đầu với zero_trust.access.applications.list. Tái hiện lệnh gọi chỉ có account_id được mô tả trong issue, sau đó xác minh rằng các requests có chính xác một tham số không còn phát sinh ValueError về tham số bị thiếu, trong khi việc cung cấp cả hai vẫn phát sinh lỗi loại trừ lẫn nhau.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Confirm this is a Python library issue and not an underlying Cloudflare API issue.
- This is an issue with the Python library
Describe the bug
In multiple classes, there is logic that attempts to extract an account_id or zone_id from the base client object if they are not provided as arguments. The code then verifies that exactly one of these parameters is present.
The current implementation looks like this:
if account_id is None:
account_id = self._client._get_account_id_path_param()
if zone_id is None:
zone_id = self._client._get_zone_id_path_param()
if account_id and zone_id:
raise ValueError("You cannot provide both account_id and zone_id")
The Bug
Both _get_account_id_path_param() and _get_zone_id_path_param() actively raise a ValueError if the client does not possess the requested parameter. This inadvertently creates a deadlock:
- If you only provide one parameter (as required), the attempt to fetch the missing second parameter from the client throws a ValueError.
- To bypass the first two if statements without throwing an error, you would have to provide both parameters (either via arguments or the client). However, doing so triggers the third if statement, throwing a
ValueErrorfor providing both.
Expected Behavior
The method should execute successfully, recognizing that exactly one mutually exclusive parameter (account_id or zone_id) was provided.
Actual Behavior
A ValueError is incorrectly raised no matter what combination of account_id or zone_id is provided.
Proposed Solution
There are two straightforward ways to resolve this deadlock:
- Wrap the parameter retrievals in a try...except block that catches and suppresses the
ValueErrors. - Modify the
_get_*_path_param()methods so they returnNoneinstead of throwing an exception when a parameter is missing.
To Reproduce
I personally encountered this issue using client.zero_trust.access.applications.list(account_id=account_id), but a quick search indicates this pattern appears in roughly 180 occurrences across the codebase.
- Initalize the Cloudflare client with an API key
- Retrieve the target account ID via searching or just first available, e.g.
account_id = next(tier(client.accounts.list())).id - Attempt to fetch the applications list using only the
account_id:applications = client.zero_trust.access.applications.list(account_id=account_id) ValueErrorincorrectly raised:Missing zone_id argument; Please provide it at the client level, e.g. Cloudflare(zone_id='abcd') or per method.
Code snippets
OS
macOS 15.7.4
Python version
3.13.11
Library version
5.0.0b2
- Ngôn ngữ chính
- Python
- Star
- 509
- Fork
- 150
- Merge trung bình
- 3 giờ 15 phút
- Pull request đã merge (30 ngày)
- 1
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 cloudflare/cloudflare-python
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
cloudflare/cloudflare-python#2747 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
cloudflare/cloudflare-python#2718 · 1 bình luận ·
-
`client.pages.projects.list` returns a paginated list of `Deployment`s instead of `Project`s Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
cloudflare/cloudflare-python#2692 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
cloudflare/cloudflare-python#2679 · 4 bình luận · 5 reaction ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
cloudflare/cloudflare-python#2745 · 1 bình luận ·
Tất cả issue của cloudflare/cloudflare-python
Issue tương tự
-
documentation help wanted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
simonw/sqlite-utils#872 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100