cloudinary / cloudinary/cloudinary_gem

CloudinaryException is too vague

Abierto
#338 3 comentarios 1 reacción 0 asignados Ver en GitHub
enhancement
Lenguaje dominante
Ruby
Estrellas
420
Forks
285
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

We've seen messages like `Resource not found` and `Error in loading`, which we'd like to handle as generic HTTP client errors (400-499). Unfortunately, we have to rescue the `CloudinaryException`, check its error message, and then determine what should be done.

I would propose having more than one exception class to make it possible for us to rescue something like `Cloudinary::HttpClientError` to ignore errors like HTTP 403 (Forbidden), 404 (Not Found), 408 (Request Timeout), etc.

One way to do this would be to add `class Cloudinary::HttpClientError < CloudinaryException; end` and then use that error for the cases that represent HTTP 400-499.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Localiza CloudinaryException y el manejo de las respuestas HTTP que actualmente produce mensajes como "Resource not found" y "Error in loading.". Rastrea cómo se clasifican las respuestas 400–499, luego define la jerarquía de excepciones y verifica que los errores de cliente relevantes se puedan capturar por separado. El issue no menciona archivos ni pruebas específicos, por lo que primero se debe identificar la cobertura existente de pruebas de excepciones y del cliente HTTP.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
ruby
Área
api, backend
Tipo de issue
Nueva funcionalidad
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.