cloudinary / cloudinary/cloudinary_gem

CloudinaryException is too vague

Ouverte
#338 3 commentaires 1 réaction 0 personnes assignées Voir sur GitHub
enhancement
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.