Log failed Preflight check at INFO level
- Ngôn ngữ chính
- Python
- Star
- 459
- Fork
- 223
- Merge trung bình
- 8 giờ 57 phút
- Pull request đã merge (30 ngày)
- 13
Mô tả
### Is your feature request related to a problem? Please describe.
I don't think the preflight check, which throws a 409 when a file exists, should be logging a WARNING in the logs.
### Describe the solution you'd like
I think Preflight in particular should log with INFO level. Although I can understand the decision for Preflight to throw a 409, in this case I am calling it specifically to ask "can I overwrite this file?" So the response, whatever it is, shouldn't be a WARNING for my application.
Put another way: If the Preflight just throws a 409 and logs a warning, I don't see how that is different to simply calling folder.upload() and letting it throw the exception. Aren't I just wasting an API call?
### Describe alternatives you've considered
I really don't want to set the SDK log level to ERROR and above, but that is the only option I think?
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu bằng cách xác định phần triển khai Preflight và log cảnh báo trong Python SDK. Xem xét các bài kiểm thử hiện có cho những lần kiểm tra Preflight thất bại, sau đó xác minh rằng phản hồi 409 vẫn được trả về, trong khi mục log tương ứng sử dụng INFO thay vì WARNING.
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
- api
- Loại issue
- Tính năng
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 42/100