Merge pull request #191 from bsara/default-gql-err-handler-log-error

separated out logging of errors to separate overriddable method in DefaultGraphQLErrorHandler

Author: oliemansm

src/main/java/graphql/servlet/DefaultGraphQLErrorHandler.java

@@ -26,20 +26,22 @@ public List<GraphQLError> processErrors(List<GraphQLError> errors) {
 
             errors.stream()
                     .filter(error -> !isClientError(error))
-                    .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());
-                        }
-                    });
+                    .forEach(this::logError);
         }
 
         return clientErrors;
     }
 
+    protected void logError(GraphQLError 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());
+        }
+    }
+
     protected List<GraphQLError> filterGraphQLErrors(List<GraphQLError> errors) {
         return errors.stream()
                 .filter(this::isClientError)