Upload API throws RuntimeException
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 20/100
Piste de recherche
Commencez par localiser la méthode d’upload dont la signature déclare IOException et suivez les cas qui lèvent actuellement RuntimeException. L’issue laisse ouverte la conception finale de la gestion des erreurs ; sa réalisation nécessite un mécanisme d’erreur cohérent et identifiable par l’appelant, dont le comportement est documenté et vérifié pour les cas concernés.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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 !).
- Langage dominant
- Java
- Étoiles
- 176
- Forks
- 113
- Merge moyen
- 9 j 20 h
- PR mergées (30 j)
- 2
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Aucun guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de cloudinary/cloudinary_java
-
`options` map modified directlyPeut-être à nouveau libre @adimiz1 l’a pris il y a 481 jours, et aucune pull request n’est ouverte. Ouverte
cloudinary/cloudinary_java#383 · 3 commentaires · 1 réaction · 1 personne assignée ·
-
Api#resourceByAssetID does not support same options as in documentationPeut-être à nouveau libre @adimiz1 l’a pris il y a 646 jours, et aucune pull request n’est ouverte. Ouverte
cloudinary/cloudinary_java#357 · 2 commentaires · 1 personne assignée ·
-
Readme - Maven dependencyPeut-être à nouveau libre @adimiz1 l’a pris il y a 754 jours, et aucune pull request n’est ouverte. Ouverte
cloudinary/cloudinary_java#349 · 1 commentaire · 1 personne assignée ·
-
cloudinary-http5 ignores system propertiesPeut-être à nouveau libre @adimiz1 l’a pris il y a 754 jours, et aucune pull request n’est ouverte. Ouverte
cloudinary/cloudinary_java#348 · 2 commentaires · 1 personne assignée ·
-
Make usage of http clients (apache, OkHttp, Jetty, Asynch) easierPeut-être à nouveau libre @ManishS6 l’a pris il y a 772 jours, et aucune pull request n’est ouverte. Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
cloudinary/cloudinary_java#343 · 6 commentaires ·
Toutes les issues de cloudinary/cloudinary_java
Issues similaires
-
bug(core): skip_nodes.json is missing SELECT and PRE, causing Python and Java to insert separators into dropdown options and preformatted textPeut-être pris @Bhumit267 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
ExceptionThrowShouldUseContext: NullPointerException on a catch block inside an annotation typePeut-être pris @Carbon14-48 l’a pris aujourd’hui. Ouverteapproved
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
checkstyle/checkstyle#22784 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
ASSERT-KTH/depclean#591 ·
Les mainteneurs répondent en général sous 1 jour
-
Api537FlareFlameModel.setStackHeightM accepts a negative heightPeut-être pris @EvenSol l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour