Merge pull request #97 from osi/patch-1

oliemansm · web-flow · commit 79c6ae21ffd8 · 2018-09-05T08:12:24.000+02:00
log ExceptionWhileDataFetching stack trace
diff --git a/src/main/java/graphql/servlet/DefaultGraphQLErrorHandler.java b/src/main/java/graphql/servlet/DefaultGraphQLErrorHandler.java
@@ -28,6 +28,8 @@ public List<GraphQLError> processErrors(List<GraphQLError> errors) {
                 .forEach(error -> {
                     if(error instanceof Throwable) {
                         log.error("Error executing query!", (Throwable) error);
+                    } else if (error instanceof ExceptionWhileDataFetching) {
+                        log.error("Error executing query {}", error.getMessage(), ((ExceptionWhileDataFetching) error).getException());
                     } else {
                         log.error("Error executing query ({}): {}", error.getClass().getSimpleName(), error.getMessage());
                     }

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,8 @@ public List<GraphQLError> processErrors(List<GraphQLError> errors) {`
`28`	`28`	`.forEach(error -> {`
`29`	`29`	`if(error instanceof Throwable) {`
`30`	`30`	`log.error("Error executing query!", (Throwable) error);`
	`31`	`+ } else if (error instanceof ExceptionWhileDataFetching) {`
	`32`	`+ log.error("Error executing query {}", error.getMessage(), ((ExceptionWhileDataFetching) error).getException());`
`31`	`33`	`} else {`
`32`	`34`	`log.error("Error executing query ({}): {}", error.getClass().getSimpleName(), error.getMessage());`
`33`	`35`	`}`