Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/779#discussion_r142407343
--- Diff:
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/RestExceptionHandler.java
---
@@ -35,7 +36,7 @@
@ResponseBody
ResponseEntity<?> handleControllerException(HttpServletRequest request,
Throwable ex) {
HttpStatus status = getStatus(request);
- return new ResponseEntity<>(new RestError(status.value(),
ex.getMessage(), getFullMessage(ex)), status);
+ return new ResponseEntity<>(new RestError(status.value(),
ex.getMessage(), ExceptionUtils.getStackTrace(ex)), status);
--- End diff --
Well, the stack trace is generally more helpful in debugging than just the
root cause, so yeah, I kinda wanted the stack trace along with the root cause.
I'm willing to be argued with, but as a dev, I'd like to know line numbers of
failures along with why.
---
