Merge pull request #219 from Kitzanos/remove-stack-trace-prod
Cherry-pick (Removed stack trace code for few exceptions)
This commit is contained in:
@@ -61,7 +61,7 @@ public class UserActionAspect {
|
|||||||
handleSuccessResponse((ResponseEntity<?>) result, storedUserActionId == null ? userActionId : storedUserActionId);
|
handleSuccessResponse((ResponseEntity<?>) result, storedUserActionId == null ? userActionId : storedUserActionId);
|
||||||
}
|
}
|
||||||
} catch (Exception ex) {
|
} catch (Exception ex) {
|
||||||
log.error("Exception occurred: ", ex);
|
// log.error("Exception occurred: ", ex);
|
||||||
handleError(ex, getUserActionIdFromRequest(request));
|
handleError(ex, getUserActionIdFromRequest(request));
|
||||||
throw ex;
|
throw ex;
|
||||||
} finally {
|
} finally {
|
||||||
|
|||||||
@@ -36,7 +36,7 @@ public class GlobalExceptionHandler {
|
|||||||
@ResponseBody
|
@ResponseBody
|
||||||
public Response<Object> handleCustomValidationException(final CustomValidationException ex) {
|
public Response<Object> handleCustomValidationException(final CustomValidationException ex) {
|
||||||
log.error(ex.getMessage());
|
log.error(ex.getMessage());
|
||||||
log.error(ex.getLocalizedMessage(), ex);
|
// log.error(ex.getLocalizedMessage(), ex);
|
||||||
return new Response<>(null, ex.getStatus(), ex.getMessage());
|
return new Response<>(null, ex.getStatus(), ex.getMessage());
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -45,14 +45,14 @@ public class GlobalExceptionHandler {
|
|||||||
@ResponseBody
|
@ResponseBody
|
||||||
public Response<Object> handleValidationException(final ValidationException ex) {
|
public Response<Object> handleValidationException(final ValidationException ex) {
|
||||||
log.error(ex.getMessage());
|
log.error(ex.getMessage());
|
||||||
log.error(ex.getLocalizedMessage(), ex);
|
// log.error(ex.getLocalizedMessage(), ex);
|
||||||
return new Response<>(ex.getErrors(), ex.getStatus(), ex.getMessage());
|
return new Response<>(ex.getErrors(), ex.getStatus(), ex.getMessage());
|
||||||
}
|
}
|
||||||
|
|
||||||
@ExceptionHandler(ResourceNotFoundException.class)
|
@ExceptionHandler(ResourceNotFoundException.class)
|
||||||
public ResponseEntity<Response<Void>> handleResourceNotFoundException(ResourceNotFoundException ex) {
|
public ResponseEntity<Response<Void>> handleResourceNotFoundException(ResourceNotFoundException ex) {
|
||||||
log.error(ex.getMessage());
|
log.error(ex.getMessage());
|
||||||
log.error(ex.getLocalizedMessage(), ex);
|
// log.error(ex.getLocalizedMessage(), ex);
|
||||||
return ResponseEntity.status(HttpStatus.NOT_FOUND)
|
return ResponseEntity.status(HttpStatus.NOT_FOUND)
|
||||||
.body(new Response<>(null, ex.getStatus(), ex.getMessage()));
|
.body(new Response<>(null, ex.getStatus(), ex.getMessage()));
|
||||||
}
|
}
|
||||||
@@ -71,7 +71,7 @@ public class GlobalExceptionHandler {
|
|||||||
@ResponseBody
|
@ResponseBody
|
||||||
public Response<Object> handleInvalidTokenException(final Throwable ex) {
|
public Response<Object> handleInvalidTokenException(final Throwable ex) {
|
||||||
log.error(ex.getMessage());
|
log.error(ex.getMessage());
|
||||||
log.error(ex.getLocalizedMessage(), ex);
|
// log.error(ex.getLocalizedMessage(), ex);
|
||||||
return new Response<>(ex.getMessage(), Status.UNAUTHORIZED, Translator.toLocale("invalid_signature"));
|
return new Response<>(ex.getMessage(), Status.UNAUTHORIZED, Translator.toLocale("invalid_signature"));
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -80,7 +80,7 @@ public class GlobalExceptionHandler {
|
|||||||
@ResponseBody
|
@ResponseBody
|
||||||
public Response<Object> unauthorizedAccessException(final Throwable ex) {
|
public Response<Object> unauthorizedAccessException(final Throwable ex) {
|
||||||
log.error(ex.getMessage());
|
log.error(ex.getMessage());
|
||||||
log.error(ex.getLocalizedMessage(), ex);
|
// log.error(ex.getLocalizedMessage(), ex);
|
||||||
return new Response<>(null, Status.UNAUTHORIZED, ex.getMessage());
|
return new Response<>(null, Status.UNAUTHORIZED, ex.getMessage());
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -115,7 +115,7 @@ public class GlobalExceptionHandler {
|
|||||||
@ResponseBody
|
@ResponseBody
|
||||||
public Response<Object> handleUnAuthorizedException(final Throwable ex) {
|
public Response<Object> handleUnAuthorizedException(final Throwable ex) {
|
||||||
log.error(ex.getMessage());
|
log.error(ex.getMessage());
|
||||||
log.error(ex.getLocalizedMessage(), ex);
|
// log.error(ex.getLocalizedMessage(), ex);
|
||||||
return new Response<>(null, Status.VALIDATION_ERROR, Translator.toLocale("invalid_login"));
|
return new Response<>(null, Status.VALIDATION_ERROR, Translator.toLocale("invalid_login"));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user