cloudfoundry / cloudfoundry/cf-java-client
Display the code passed to UaaException in UaaException.getMessage()
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Java
- Estrellas
- 334
- Forks
- 319
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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)
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con el constructor de UaaException mostrado en el issue y, a continuación, inspecciona las otras subclases concretas de AbstractCloudFoundryException, incluidas ClientV2Exception, Clientv3Exception y UnknownCloudFoundryException. Confirma que los mensajes de las excepciones exponen el código de estado de forma coherente, tomando como referencia las clases que ya son correctas, y verifica el comportamiento resultante con las pruebas existentes del proyecto.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java
- Área
- api, backend
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100