Upload API throws RuntimeException
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 20/100
Hướng nghiên cứu
Bắt đầu bằng cách xác định phương thức upload có chữ ký khai báo IOException và lần theo các trường hợp hiện đang ném RuntimeException. Issue để ngỏ thiết kế xử lý lỗi cuối cùng; để được xem là hoàn tất, cần có một cơ chế lỗi nhất quán mà caller có thể nhận diện, với hành vi của cơ chế đó được ghi lại và xác minh cho các trường hợp bị ảnh hưởng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Hello,
You should provide an easy way to handle upload errors in your SDK.
The method signature suggests that you will be throwing an IOException but looking at the source code, we can see that you're actually throwing RuntimeException in several cases ! These cases should be handled differently in the SDK so that the caller can identify them : they are definitely not a reason to throw a RuntimeException.
At the minimum, you should throw a IOException as specified by the upload signature.
Alternatively, you could think about a nicer error handling system, either with a custom Exception or with a specific Map returned (whose values should be documented !).
- Ngôn ngữ chính
- Java
- Star
- 176
- Fork
- 113
- Merge trung bình
- 9 ngày 20 giờ
- Pull request đã merge (30 ngày)
- 2
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Không 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 cloudinary/cloudinary_java
-
`options` map modified directlyCó thể làm lại được @adimiz1 đã nhận 481 ngày trước và không có pull request nào đang mở. Đang mở
cloudinary/cloudinary_java#383 · 3 bình luận · 1 reaction · 1 người được giao ·
-
Api#resourceByAssetID does not support same options as in documentationCó thể làm lại được @adimiz1 đã nhận 646 ngày trước và không có pull request nào đang mở. Đang mở
cloudinary/cloudinary_java#357 · 2 bình luận · 1 người được giao ·
-
Readme - Maven dependencyCó thể làm lại được @adimiz1 đã nhận 754 ngày trước và không có pull request nào đang mở. Đang mở
cloudinary/cloudinary_java#349 · 1 bình luận · 1 người được giao ·
-
cloudinary-http5 ignores system propertiesCó thể làm lại được @adimiz1 đã nhận 754 ngày trước và không có pull request nào đang mở. Đang mở
cloudinary/cloudinary_java#348 · 2 bình luận · 1 người được giao ·
-
Make usage of http clients (apache, OkHttp, Jetty, Asynch) easierCó thể làm lại được @ManishS6 đã nhận 772 ngày trước và không có pull request nào đang mở. Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
cloudinary/cloudinary_java#343 · 6 bình luận ·
Tất cả issue của cloudinary/cloudinary_java
Issue tương tự
-
bug(core): skip_nodes.json is missing SELECT and PRE, causing Python and Java to insert separators into dropdown options and preformatted textCó thể đã có người làm @Bhumit267 đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
Maintainer thường phản hồi trong vòng 1 ngày
-
ExceptionThrowShouldUseContext: NullPointerException on a catch block inside an annotation typeCó thể đã có người làm @Carbon14-48 đã nhận hôm nay. Đang mởapproved
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
checkstyle/checkstyle#22784 · 1 bình luận ·
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 74/100
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
ASSERT-KTH/depclean#591 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Api537FlareFlameModel.setStackHeightM accepts a negative heightCó thể đã có người làm @EvenSol đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
Maintainer thường phản hồi trong vòng 1 ngày