Cannot override SimpleDataFetcherExceptionHandler

Hi, Thank you for this great implementation. I am using version 7.0.1. Facing problem with exception handling. Tried lots of different ways. like this one: https://github.com/leangen/graphql-spqr/issues/219

`@Configuration public class GraphQLConfig {

@Bean public GraphQL graphQL(GraphQLSchema schema) { return GraphQL.newGraphQL(schema) .queryExecutionStrategy(new AsyncExecutionStrategy(new CustomExceptionHandler())) .mutationExecutionStrategy(new AsyncSerialExecutionStrategy(new CustomExceptionHandler())) .build(); } }Also have public class CustomExceptionHandler implements DataFetcherExceptionHandler {

@Override public DataFetcherExceptionHandlerResult onException( DataFetcherExceptionHandlerParameters handlerParameters) {

Throwable exception = handlerParameters.getException();

// do something with exception
System.out.println("asdadad");

GraphQLError error = GraphqlErrorBuilder.newError().message(exception.getMessage()).build();

return DataFetcherExceptionHandlerResult.newResult().error(error).build();

} }`

My CustomExceptionHandler is not triggering. Please, help to solve this issue. P.S: I am new in GraphQL, also sorry my english.

[erdenebadral]

Thank you very much. If someone looking code example:

public class CustomGraphQLError implements GraphQLError {

  private static final long serialVersionUID = 1L;

  private String message;
  private final Map<String, Object> extensions;

  public CustomGraphQLError(String message, Map<String, Object> extensions) {
    this.extensions = extensions;
    this.message = message;
  }
  @Override
  public String getMessage() {
    return message;
  }

  @Override
  public List<SourceLocation> getLocations() {
    return null;
  }

  @Override
  public ErrorClassification getErrorType() {
    return null;
  }

  @Override
  public Map<String, Object> getExtensions() {
    return extensions;
  }

}

@Component
public class GraphQLExceptionHandler {

  @ExceptionHandler(CustomValidationException.class)
  GraphQLError myCustomHandler(CustomValidationException e) {
    CustomGraphQLError customError = new CustomGraphQLError(e.getMessage(), e.getExtensions());
    return customError;
  }

}

public class CustomValidationException extends RuntimeException implements GraphQLError {

  private static final long serialVersionUID = 1L;
  private final HashMap<String, Object> extensions;

  public CustomValidationException(Map<String, String> message) {
    super("Validation Error");
    extensions = new HashMap<String, Object>();
    extensions.putAll(message);
  }

  @Override
  public List<SourceLocation> getLocations() {    
    return null;
  }

  @Override
  public ErrorType getErrorType() {
    return null;
  }

  @Override
  public Map<String, Object> getExtensions() {
    return extensions;
  }

}

@Transactional
  public Book addBook(BookInput input) {
    ValidatorFactory factory = Validation.buildDefaultValidatorFactory();
    Validator validator = factory.getValidator();

    Book book =
        Book.builder().name(input.getName()).publishedDate(input.getPublishedDate()).build();
    Set<ConstraintViolation<Book>> constraintViolations = validator.validate(book);
    if (constraintViolations.isEmpty()) {
      return bookRepository.save(book);
    } else {
      Map<String, String> message = new HashMap<String, String>();
      for (ConstraintViolation<Book> contraints : constraintViolations) {
        message.put(contraints.getPropertyPath().toString(), contraints.getMessage());
      }
      throw new CustomValidationException(message);
    }

  }

[oliemansm]

You could create a custom implementation of GraphQLError and return that. If you return null for the locations and just limit the values returned by getExtensions() you should get the result you’re after.