Upload API throws RuntimeException
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 20/100
Rechercherichtung
Beginne damit, die Upload-Methode zu finden, deren Signatur IOException deklariert, und verfolge die Fälle, die derzeit RuntimeException auslösen. Das Issue lässt das endgültige Design der Fehlerbehandlung offen; abgeschlossen ist die Aufgabe, wenn ein konsistenter, vom Aufrufer identifizierbarer Fehlermechanismus vorhanden ist und sein Verhalten für die betroffenen Fälle dokumentiert und verifiziert wurde.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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 !).
- Vorherrschende Sprache
- Java
- Sterne
- 176
- Forks
- 113
- Ø Merge
- 9 T. 20 Std.
- Gemergte PRs (30 T.)
- 2
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Kein Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus cloudinary/cloudinary_java
-
`options` map modified directlyEvtl. wieder frei @adimiz1 hat das vor 481 Tagen übernommen, und es ist kein Pull Request offen. Offen
cloudinary/cloudinary_java#383 · 3 Kommentare · 1 Reaktion · 1 zugewiesene Person ·
-
Api#resourceByAssetID does not support same options as in documentationEvtl. wieder frei @adimiz1 hat das vor 646 Tagen übernommen, und es ist kein Pull Request offen. Offen
cloudinary/cloudinary_java#357 · 2 Kommentare · 1 zugewiesene Person ·
-
Readme - Maven dependencyEvtl. wieder frei @adimiz1 hat das vor 754 Tagen übernommen, und es ist kein Pull Request offen. Offen
cloudinary/cloudinary_java#349 · 1 Kommentar · 1 zugewiesene Person ·
-
cloudinary-http5 ignores system propertiesEvtl. wieder frei @adimiz1 hat das vor 754 Tagen übernommen, und es ist kein Pull Request offen. Offen
cloudinary/cloudinary_java#348 · 2 Kommentare · 1 zugewiesene Person ·
-
Make usage of http clients (apache, OkHttp, Jetty, Asynch) easierEvtl. wieder frei @ManishS6 hat das vor 772 Tagen übernommen, und es ist kein Pull Request offen. Offen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
cloudinary/cloudinary_java#343 · 6 Kommentare ·
Alle Issues in cloudinary/cloudinary_java
Ähnliche Issues
-
bug(core): skip_nodes.json is missing SELECT and PRE, causing Python and Java to insert separators into dropdown options and preformatted textEvtl. vergeben @Bhumit267 hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 1 Tag
-
ExceptionThrowShouldUseContext: NullPointerException on a catch block inside an annotation typeEvtl. vergeben @Carbon14-48 hat das heute übernommen. Offenapproved
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
checkstyle/checkstyle#22784 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
ASSERT-KTH/depclean#591 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Api537FlareFlameModel.setStackHeightM accepts a negative heightEvtl. vergeben @EvenSol hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
Maintainer antworten meist innerhalb von 1 Tag