cloudfoundry / cloudfoundry/cf-java-client
Display the code passed to UaaException in UaaException.getMessage()
Personne n'a encore pris cette issue.
- Langage dominant
- Java
- Étoiles
- 334
- Forks
- 319
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
Make the logging more informative for the developer. Change simply the code from
public UaaException(Integer statusCode, String error, String errorDescription) {
super(statusCode, String.format("%s: %s", error, errorDescription));
this.error = error;
...
to
public UaaException(Integer statusCode, String error, String errorDescription) {
super(statusCode, String.format("%s: %s [%s]", error, errorDescription, statusCode));
this.error = error;
...
This will avoid painful debugging and work around.
Do the same for the other concrete classes extending AbstractCloudFoundryException aka ClientV2Exception (seems good), Clientv3Exception (seems good), UnknownCloudFoundryException (to fix)
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par le constructeur de UaaException présenté dans l’issue, puis examinez les autres sous-classes concrètes de AbstractCloudFoundryException, notamment ClientV2Exception, Clientv3Exception et UnknownCloudFoundryException. Confirmez que les messages des exceptions exposent systématiquement le code d’état, en prenant comme références les classes déjà correctes, et vérifiez le comportement obtenu avec les tests existants du projet.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java
- Domaine
- api, backend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100