From 3ddfac6165e58dc2d9ae7fd150f89bf41c551cc6 Mon Sep 17 00:00:00 2001 From: Federico Rispo Date: Sat, 27 Jul 2024 08:04:41 +0200 Subject: [PATCH] fix: remove Jdk8Module Jackson module registration 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 --- .../error/DefaultGraphQLServletObjectMapperConfigurer.java | 3 --- 1 file changed, 3 deletions(-) diff --git a/graphql-java-kickstart/src/main/java/graphql/kickstart/execution/error/DefaultGraphQLServletObjectMapperConfigurer.java b/graphql-java-kickstart/src/main/java/graphql/kickstart/execution/error/DefaultGraphQLServletObjectMapperConfigurer.java index 53d71288..40d4bfa4 100644 --- a/graphql-java-kickstart/src/main/java/graphql/kickstart/execution/error/DefaultGraphQLServletObjectMapperConfigurer.java +++ b/graphql-java-kickstart/src/main/java/graphql/kickstart/execution/error/DefaultGraphQLServletObjectMapperConfigurer.java @@ -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 */ @@ -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); } }