Nike-Inc/backstopper

Include ApiException cause info in the backstopper log message

nicmunroe opened this issue · 1 comments

Currently, GenericApiExceptionHandlerListener includes a api_exception_message=ApiException.getMessage() key/value pair for the logs. But there's no info by default on the ApiException.getCause().

This is potentially useful info when the ApiError(s) in the ApiException are too generic, the thrower of the ApiException doesn't include any extra logging details, and the whole thing ultimately maps to a 4xx (and therefore no stack trace shows up in the logs).

We should add exception_cause_class and exception_cause_message log pairs in GenericApiExceptionHandlerListener. That will result in the exception cause's class and message always showing up in the logs.

Fixed by PR #46, released in version v0.13.0.