Skip to content

Commit

Permalink
fix: remove Jdk8Module Jackson module registration (#574)
Browse files Browse the repository at this point in the history
Since the project now runs on java >= 11 and uses the latest jackson
version, it is no longer necessary to register this module because
it is included in jackson-databind.

Refs: #570
  • Loading branch information
federicorispo committed Sep 13, 2024
1 parent 495e302 commit 74cd2d0
Showing 1 changed file with 0 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
import com.fasterxml.jackson.annotation.JsonInclude;
import com.fasterxml.jackson.databind.ObjectMapper;
import com.fasterxml.jackson.databind.SerializationFeature;
import com.fasterxml.jackson.datatype.jdk8.Jdk8Module;
import graphql.kickstart.execution.config.GraphQLServletObjectMapperConfigurer;

/** @author Andrew Potter */
Expand All @@ -12,9 +11,7 @@ public class DefaultGraphQLServletObjectMapperConfigurer

@Override
public void configure(ObjectMapper mapper) {
// default configuration for GraphQL Java Servlet
mapper.disable(SerializationFeature.FAIL_ON_EMPTY_BEANS);
mapper.registerModule(new Jdk8Module());
mapper.setDefaultPropertyInclusion(JsonInclude.Include.ALWAYS);
}
}

0 comments on commit 74cd2d0

Please sign in to comment.