set_properties() silently ignores per-property PROPPATCH failures (regression since v3.0)
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ó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 78/100
Hướng nghiên cứu
Bắt đầu trong davobject.py tại DAVObject.set_properties() và _post_set_properties, sau đó kiểm tra DAVResponse.all_statuses_ok() cùng các bài kiểm thử đơn vị hiện có cho việc cập nhật thuộc tính. Tái hiện vấn đề bằng một phản hồi 207 có trạng thái 403 cho một thuộc tính. Hoàn tất khi set_properties() báo lỗi đó và có bài kiểm thử đơn vị bao quát phản hồi này.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Disclaimer: AI-generated issue, written by Claude Opus 5.5 (Claude Code). I wanted to delete some old code review documents, but apparently they still contained some unhandled findings.
Since v3.0, DAVObject.set_properties() (davobject.py:590) only raises PropsetError when the HTTP status is >= 400. A server that answers 207 Multi-Status with a per-property 403 Forbidden or 409 (e.g. a read-only or protected property) is reported as success. v2.2.6 checked every DAV:status in the reply.
Repro: a 207 reply with <propstat><prop><displayname/></prop><status>HTTP/1.1 403 Forbidden</status></propstat>, then set_properties([dav.DisplayName("x")]) returns normally. Fix: use DAVResponse.all_statuses_ok() (or the per-propstat check) in _post_set_properties, and add a unit test.
- Ngôn ngữ chính
- Python
- Star
- 412
- Fork
- 113
- Merge trung bình
- 11 giờ 37 phút
- Pull request đã merge (30 ngày)
- 50
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không 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 python-caldav/caldav
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
python-caldav/caldav#799 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
python-caldav/caldav#788 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
python-caldav/caldav#687 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 75/100
python-caldav/caldav#796 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 66/100
python-caldav/caldav#795 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của python-caldav/caldav
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
-
[Device Support Request] HOBEIAN ZG-204ZL (Tuya PIR motion sensor with illuminance)Có thể đã có người làm @lenucksi đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
zigpy/zha-device-handlers#5493 ·
Maintainer thường phản hồi trong vòng 3 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
drevendev/BeautifulRepo#42 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
writerslogic/holographic-memory#93 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
antoine-tena/kiln#47 ·
Maintainer thường phản hồi trong vòng 1 ngày