Upload API throws RuntimeException
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 20/100
Línea de trabajo
Empieza por localizar el método de carga cuya firma declara IOException y rastrea los casos que actualmente lanzan RuntimeException. El issue deja abierto el diseño final del manejo de errores; para darlo por terminado se requiere un mecanismo de error coherente e identificable por el llamador, con su comportamiento documentado y verificado para los casos afectados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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 !).
- Lenguaje dominante
- Java
- Estrellas
- 176
- Forks
- 113
- Merge medio
- 9 d 20 h
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cloudinary/cloudinary_java
-
`options` map modified directlyQuizá libre de nuevo @adimiz1 la tomó hace 481 días y no hay ningún pull request abierto. Abierto
cloudinary/cloudinary_java#383 · 3 comentarios · 1 reacción · 1 asignado ·
-
Api#resourceByAssetID does not support same options as in documentationQuizá libre de nuevo @adimiz1 la tomó hace 646 días y no hay ningún pull request abierto. Abierto
cloudinary/cloudinary_java#357 · 2 comentarios · 1 asignado ·
-
Readme - Maven dependencyQuizá libre de nuevo @adimiz1 la tomó hace 754 días y no hay ningún pull request abierto. Abierto
cloudinary/cloudinary_java#349 · 1 comentario · 1 asignado ·
-
cloudinary-http5 ignores system propertiesQuizá libre de nuevo @adimiz1 la tomó hace 754 días y no hay ningún pull request abierto. Abierto
cloudinary/cloudinary_java#348 · 2 comentarios · 1 asignado ·
-
Make usage of http clients (apache, OkHttp, Jetty, Asynch) easierQuizá libre de nuevo @ManishS6 la tomó hace 772 días y no hay ningún pull request abierto. Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
cloudinary/cloudinary_java#343 · 6 comentarios ·
Todos los issues de cloudinary/cloudinary_java
Issues similares
-
bug(core): skip_nodes.json is missing SELECT and PRE, causing Python and Java to insert separators into dropdown options and preformatted textPosiblemente ocupada @Bhumit267 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
ExceptionThrowShouldUseContext: NullPointerException on a catch block inside an annotation typePosiblemente ocupada @Carbon14-48 la tomó hoy. Abiertoapproved
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
checkstyle/checkstyle#22784 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
ASSERT-KTH/depclean#591 ·
Los mantenedores suelen responder en 1 día
-
Api537FlareFlameModel.setStackHeightM accepts a negative heightPosiblemente ocupada @EvenSol la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día