cloudinary / cloudinary/cloudinary_gem
CloudinaryException is too vague
- 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
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