cloudinary / cloudinary/cloudinary_gem
CloudinaryException is too vague
- Langage dominant
- Ruby
- Étoiles
- 420
- Forks
- 285
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Localisez CloudinaryException et la gestion des réponses HTTP qui produit actuellement des messages tels que "Resource not found" et "Error in loading.". Retracez la manière dont les réponses 400–499 sont classifiées, puis définissez la hiérarchie des exceptions et vérifiez que les erreurs client pertinentes peuvent être interceptées séparément. L’issue ne mentionne pas de fichiers ni de tests spécifiques ; il faut donc d’abord identifier la couverture existante des tests d’exceptions et du client HTTP.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- ruby
- Domaine
- api, backend
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100