Github user merrimanr commented on a diff in the pull request:
https://github.com/apache/metron/pull/779#discussion_r142237205
--- 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 --
I would prefer just root cause but I had the impression the full stack
trace was wanted. Doesn't make that much difference to me so we just need to
agree on what it should be.
---
