błędna obsługa wyjątków

[michbudz]

Po wprowadzeniu zmian z obsługą wyjątków w wersji 3.0.24 powstała niezgodność.

Dla błędu HTTP 400, exceptionCode: 21301 brak informacji o szczegółach błędu w obiekcie ApiException i BadRequestApiException.

Scenariusz:

1. Biblioteka w wersji 3.0.24
2. Nie ustawiam domyślnego nagłówka X-Error-Format: "problem-details" - chcę, aby biblioteka działała po staremu.
3. Symuluję błąd HTTP 400, exceptionCode: 21301 poprzez dwukrotne wywołanie API redeemToken
4. Otrzymuję wyjątek BadRequestApiException, który nie zawiera szczegółowych informacji o kodzie 21301

Tutaj podgląd informacji o wyjątku:

Przed wprowadzeniem zmian w wersji 3.0.23 szczegółowe informacje były zawarte w ApiException -> ExceptionResponse -> ExceptionObject -> ExceptionDetails

Według mnie jest to BC. Proszę o opinię.

[AK47454453]

@michbudz
Czy aby na pewno w żądaniu nie jest wysyłany ten nagłówek? (na screenie widać tylko te z odpowiedzi) - jeśli używasz modułu demo-web-app to tam do każdego application.yamla został dodany ten nagłówek w default-headers dlatego też jest wymuszone nowe zachowanie - oczywiście można to sobie modyfikować, bo to w końcu demo/test sdk clienta.
Dla przykładu do tego testu pl.akmf.ksef.sdk.KsefTokenIntegrationTest#tokenTest na koniec metody można dodać jeszcze raz wywołanie redeemToken aby mieć dokładnie ten sam scenariusz (poniżej kawałek kodu) i wszystkie poniższe assercje są spełnione, a więc działa 'po staremu' - oczywiście wcześniej wyrzucając z application.yaml wspomniany wyżej nagłówek.

` ApiException apiException = Assertions.assertThrows(ApiException.class, () ->
ksefClient.redeemToken(response.getAuthenticationToken().getToken()));

    KsefApiException ksefApiException = (KsefApiException) apiException;
    Assertions.assertNotNull(ksefApiException);
    Assertions.assertEquals(400, ksefApiException.getCode());
    Assertions.assertNotNull(ksefApiException.getExceptionResponse());
    ExceptionObject exceptionObject = ksefApiException.getExceptionResponse().getException();
    Assertions.assertNotNull(exceptionObject);
    Assertions.assertNotNull(exceptionObject.getTimestamp());
    Assertions.assertNotNull(exceptionObject.getServiceCode());
    List<ExceptionDetails> exceptionDetails = exceptionObject.getExceptionDetailList();
    Assertions.assertEquals(1, exceptionDetails.size());
    Assertions.assertEquals("Brak autoryzacji.", exceptionDetails.get(0).getExceptionDescription());
    Assertions.assertEquals(21301, exceptionDetails.get(0).getExceptionCode());
    List<String> details = exceptionDetails.get(0).getDetails();
    Assertions.assertEquals(1, details.size());
    Assertions.assertTrue(details.get(0)
            .contains("Tokeny dla operacji uwierzytelniania"));
    Assertions.assertTrue(details.get(0)
            .contains("zostały już pobrane."));`

[michbudz]

@AK47454453 Czy u Ciebie te testy przechodzą poprawnie? U mnie jest błąd, ponieważ drugie wywołanie redeemToken zwraca wyjątek typu BadRequestApiException.

Zrobiłem dokładnie tak:

1. Wziąłem czystą aktualną gałąź źródeł main
2. Usunąłem w application.yaml defaultowe nagłówki default-headers
3. Dodałem wskazane testy w pl.akmf.ksef.sdk.KsefTokenIntegrationTest#tokenTest

Moim zdaniem błąd zachodzi w metodzie ExceptionHandler#handleException.

1. W linii kodu 61-62 wchodzi do warunku

if (contentType.contains(APPLICATION_JSON)
                || contentType.contains(APPLICATION_PROBLEM_JSON)) {

ponieważ contentType jest ustawiony na application/json; charset=utf-8
2. Następnie jest wykonywana część

 tryParseException(() -> {
                        BadRequestProblemDetails badRequestProblemDetails = objectMapper.readValue(response.body(), BadRequestProblemDetails.class);
                        throw new BadRequestApiException(response.statusCode(), uri, method, message, response.headers(), badRequestProblemDetails);
                    });

3. W linii 86 ObjectMapper wykonuje mapowanie

BadRequestProblemDetails badRequestProblemDetails = objectMapper.readValue(response.body(), BadRequestProblemDetails.class);

na pusty obiekt BadRequestProblemDetails

badRequestProblemDetails = {BadRequestProblemDetails@17033} "title='null, status=0, instance='null, detail='null, errors=null, timestamp='null, traceId='null"
 title = null
 status = 0
 instance = null
 detail = null
 errors = null
 timestamp = null
 traceId = null

4. A następnie zwracany jest wyjątek BadRequestApiException

Fallback do poprzedniej wersji wyjątku nigdy nie jest wykonywany.

[AK47454453]

Obaj mamy rację, tzn mój test i asercje które wskazałem działają przy założeniu, że property fail_on_unknown_properties = true, w Twoim przypadku false faktycznie zwraca pusty obiekt i nie wchodzi w fallback - dzięki za uwagę, w następnej wersji dodam zabezpieczenie, planowana data wydania to przyszły wtorek/środa.