mapbox / mapbox/mapbox-java

When API request fails, log message for better troubleshooting

Ouverte
#609 5 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
Java
Étoiles
438
Forks
117
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Just spent some time figuring out why my response body was null only to learn that the request was failing with a message `Overview option must be full for congestion`. We should add a field for message in the `DirectionsResponse` for example and if that message isn't null, throw a runtime exception or at least log it.

cc: @zugaldia

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par le modèle DirectionsResponse et suivez la manière dont les échecs de l’API sont actuellement gérés lorsque le corps de la réponse est null. Confirmez comment la valeur `message` d’une requête ayant échoué doit être exposée et si elle doit être journalisée ou provoquer une exception d’exécution. Le travail est terminé lorsque des échecs tels que `Overview option must be full for congestion` ne sont plus silencieux.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
api
Type d'issue
Fonctionnalité
Difficulté
3/5
Temps estimé
1-2 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.