Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Per-request retry_params.min_wait_in_ms overwrites max_retry

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#319 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
78/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
api

Hướng nghiên cứu

Bắt đầu trong openfga_sdk/api_client.py và bản tương ứng đồng bộ của nó, đọc khối _retry_params cho từng request và hành vi của RetryParams được mô tả trong issue. Cập nhật cả hai đường thực thi để thiết lập thời gian chờ cho từng request không thay thế max_retry, sau đó kiểm thử kịch bản list_objects với max_retry=0 và xác nhận rằng các giới hạn retry cũng như thiết lập thời gian chờ hoạt động như mong đợi.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Bug

In openfga_sdk/api_client.py (and the sync twin), applying per-request _retry_params assigns min_wait_in_ms onto max_retry:

if _retry_params is not None:
    if _retry_params.max_retry is not None:
        max_retry = _retry_params.max_retry
    if _retry_params.min_wait_in_ms is not None:
        max_retry = _retry_params.min_wait_in_ms  # should be min_wait_in_ms =
    if _retry_params.max_wait_in_sec is not None:
        max_wait_in_sec = _retry_params.max_wait_in_sec

Introduced in https://github.com/openfga/python-sdk/commit/fb553501c29e38ff21d04ac4afccae83f995b6d2 (feat: improved handling of retries (#188)). Still present on main and in v0.10.4 (latest PyPI as of 2026-09-26).

RetryParams.__init__ always sets min_wait_in_ms (default 100), so any per-request RetryParams overwrites max_retry with the wait-in-ms value.

Impact

await client.list_objects(body, options={"retry_params": RetryParams(max_retry=0)})

Intended: 1 attempt (range(0 + 1)).

Actual: max_retry becomes 100 → 101 attempts on 5xx (range(100 + 1)).

Same bug in openfga_sdk/sync/api_client.py.

Expected

if _retry_params.min_wait_in_ms is not None:
    min_wait_in_ms = _retry_params.min_wait_in_ms

Workaround

Pass a duck-typed object with max_retry=0 and min_wait_in_ms=None so the broken assignment is skipped.

Ngôn ngữ chính
Python
Star
85
Fork
36
Merge trung bình
4 giờ 22 phút
Pull request đã merge (30 ngày)
2

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của openfga/python-sdk

Tất cả issue của openfga/python-sdk

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.