Log all filtered graphql errors.

Author: apottere

src/main/java/graphql/servlet/GraphQLServlet.java

@@ -21,6 +21,7 @@
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.RuntimeJsonMappingException;
 import com.fasterxml.jackson.databind.annotation.JsonDeserialize;
+import graphql.ExceptionWhileDataFetching;
 import graphql.ExecutionResult;
 import graphql.GraphQL;
 import graphql.GraphQLError;
@@ -306,6 +307,12 @@ private Map<String, Object> createResultFromDataAndErrors(Object data, List<Grap
             if (clientErrors.size() < errors.size()) {
                 // Some errors were filtered out to hide implementation - put a generic error in place.
                 clientErrors.add(new GenericGraphQLError("Internal Server Error(s) while executing query"));
+
+                errors.stream()
+                    .filter(error -> !isClientError(error))
+                    .forEach(error -> {
+                        log.error("Error executing query ({}): {}", error.getClass().getSimpleName(), error.getMessage());
+                    });
             }
             result.put("errors", clientErrors);
         }
@@ -318,9 +325,13 @@ private boolean errorsPresent(List<GraphQLError> errors) {
     }
 
     protected List<GraphQLError> filterGraphQLErrors(List<GraphQLError> errors) {
-        return errors.stream().
-                filter(error -> error instanceof InvalidSyntaxError || error instanceof ValidationError).
-                collect(Collectors.toList());
+        return errors.stream()
+            .filter(this::isClientError)
+            .collect(Collectors.toList());
+    }
+
+    protected boolean isClientError(GraphQLError error) {
+        return error instanceof InvalidSyntaxError || error instanceof ValidationError;
     }
 
     private <T> void runListeners(List<T> listeners, Consumer<? super T> action) {