Upgrade to Jackson 3 and JDK 17

Author: justin-tay

.github/workflows/ci.yml

@@ -13,7 +13,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        java: [ 8, 11, 17, 21 ]
+        java: [ 17, 21, 25 ]
     name: Java ${{ matrix.java }}
     steps:
       - uses: actions/checkout@v4

pom.xml

@@ -68,16 +68,16 @@
   </distributionManagement>
 
   <properties>
-    <java.testversion>1.8</java.testversion>
-    <java.version>1.8</java.version>
+    <java.testversion>17</java.testversion>
+    <java.version>17</java.version>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
 
     <version.itu>1.14.0</version.itu>
-    <version.jackson>2.18.3</version.jackson>
+    <version.jackson>3.0.0-rc10</version.jackson>
     <version.joni>2.2.6</version.joni>
-    <version.logback>1.3.14</version.logback> <!-- 1.4.x and above is not Java 8 compatible -->
+    <version.logback>1.5.19</version.logback>
     <version.slf4j>2.0.17</version.slf4j>
-    <version.graaljs>21.3.10</version.graaljs> <!-- 22.x and above is not Java 8 compatible -->
+    <version.graaljs>24.2.2</version.graaljs>
     <version.hamcrest>3.0</version.hamcrest>
     <version.junit>5.11.3</version.junit>
     <version.jacoco-maven-plugin>0.8.12</version.jacoco-maven-plugin>
@@ -112,13 +112,13 @@
     </dependency>
 
     <dependency>
-      <groupId>com.fasterxml.jackson.core</groupId>
+      <groupId>tools.jackson.core</groupId>
       <artifactId>jackson-databind</artifactId>
       <version>${version.jackson}</version>
     </dependency>
 
     <dependency>
-      <groupId>com.fasterxml.jackson.dataformat</groupId>
+      <groupId>tools.jackson.dataformat</groupId>
       <artifactId>jackson-dataformat-yaml</artifactId>
       <version>${version.jackson}</version>
     </dependency>
@@ -459,7 +459,7 @@
                       </exports>
                       <requires>
                         static com.ethlo.time;
-                        static com.fasterxml.jackson.dataformat.yaml;
+                        static tools.jackson.dataformat.yaml;
                         static org.jruby.jcodings;
                         static org.jruby.joni;
                         static org.graalvm.sdk;

src/main/java/com/networknt/schema/Error.java

@@ -20,9 +20,9 @@
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.JsonInclude.Include;
 import com.fasterxml.jackson.annotation.JsonPropertyOrder;
-import com.fasterxml.jackson.databind.JsonNode;
-import com.fasterxml.jackson.databind.annotation.JsonSerialize;
-import com.fasterxml.jackson.databind.ser.std.ToStringSerializer;
+import tools.jackson.databind.JsonNode;
+import tools.jackson.databind.annotation.JsonSerialize;
+import tools.jackson.databind.ser.std.ToStringSerializer;
 import com.networknt.schema.i18n.MessageFormatter;
 import com.networknt.schema.path.NodePath;
 import com.networknt.schema.utils.CachingSupplier;

src/main/java/com/networknt/schema/ErrorMessages.java

@@ -4,7 +4,7 @@
 import java.util.LinkedHashMap;
 import java.util.Map;
 
-import com.fasterxml.jackson.databind.JsonNode;
+import tools.jackson.databind.JsonNode;
 
 /**
  * ErrorMessages.
@@ -24,12 +24,12 @@ public static Map<String, String> getErrorMessage(Schema parentSchema, String er
         if (message != null) {
             JsonNode messageNode = message.get(keyword);
             if (messageNode != null) {
-                if (messageNode.isTextual()) {
-                    return Collections.singletonMap("", messageNode.asText());
+                if (messageNode.isString()) {
+                    return Collections.singletonMap("", messageNode.asString());
                 } else if (messageNode.isObject()) {
                     Map<String, String> result = new LinkedHashMap<>();
-                    messageNode.fields()
-                            .forEachRemaining(entry -> result.put(entry.getKey(), entry.getValue().textValue()));
+                    messageNode.properties().iterator()
+                            .forEachRemaining(entry -> result.put(entry.getKey(), entry.getValue().asString()));
                     if (!result.isEmpty()) {
                         return result;
                     }

src/main/java/com/networknt/schema/Schema.java

@@ -32,7 +32,7 @@
 import java.util.Objects;
 import java.util.function.Consumer;
 
-import com.fasterxml.jackson.databind.JsonNode;
+import tools.jackson.databind.JsonNode;
 import com.networknt.schema.keyword.KeywordValidator;
 import com.networknt.schema.keyword.TypeValidator;
 import com.networknt.schema.path.NodePath;
@@ -108,11 +108,11 @@ private static Schema obtainSubSchemaNode(final JsonNode schemaNode, final Schem
             return null;
         }
 
-        final String text = node.textValue();
+        final String text = node.asString();
         if (text == null) {
             return null;
         }
-        final SchemaLocation schemaLocation = SchemaLocation.of(node.textValue());
+        final SchemaLocation schemaLocation = SchemaLocation.of(node.asString());
         return schemaContext.getSchemaRegistry().getSchema(schemaLocation);
     }
     public static class JsonNodePathLegacy {
@@ -522,7 +522,7 @@ private List<KeywordValidator> read(JsonNode schemaNode) {
         } else {
             KeywordValidator refValidator = null;
 
-            Iterator<Entry<String, JsonNode>> iterator = schemaNode.fields();
+            Iterator<Entry<String, JsonNode>> iterator = schemaNode.properties().iterator();
             validators = new ArrayList<>(schemaNode.size());
             while (iterator.hasNext()) {
                 Entry<String, JsonNode> entry = iterator.next();

src/main/java/com/networknt/schema/SchemaContext.java

@@ -19,7 +19,7 @@
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.ConcurrentMap;
 
-import com.fasterxml.jackson.databind.JsonNode;
+import tools.jackson.databind.JsonNode;
 import com.networknt.schema.dialect.Dialect;
 import com.networknt.schema.keyword.KeywordValidator;
 

src/main/java/com/networknt/schema/SchemaRegistry.java

@@ -16,7 +16,7 @@
 
 package com.networknt.schema;
 
-import com.fasterxml.jackson.databind.JsonNode;
+import tools.jackson.databind.JsonNode;
 import com.networknt.schema.dialect.BasicDialectRegistry;
 import com.networknt.schema.dialect.DefaultDialectRegistry;
 import com.networknt.schema.dialect.Dialect;
@@ -538,18 +538,18 @@ protected SchemaContext createSchemaContext(final JsonNode schemaNode) {
 
     private Dialect getDialect(final JsonNode schemaNode, SchemaRegistryConfig config) {
         final JsonNode iriNode = schemaNode.get("$schema");
-        if (iriNode != null && iriNode.isTextual()) {
-            return getDialect(iriNode.textValue());
+        if (iriNode != null && iriNode.isString()) {
+            return getDialect(iriNode.asString());
         }
         return null;
     }
 
     private Dialect getDialectOrDefault(final JsonNode schemaNode) {
         final JsonNode iriNode = schemaNode.get("$schema");
-        if (iriNode != null && !iriNode.isNull() && !iriNode.isTextual()) {
+        if (iriNode != null && !iriNode.isNull() && !iriNode.isString()) {
             throw new SchemaException("Unknown dialect: " + iriNode);
         }
-        final String iri = iriNode == null || iriNode.isNull() ? defaultDialectId : iriNode.textValue();
+        final String iri = iriNode == null || iriNode.isNull() ? defaultDialectId : iriNode.asString();
         return getDialect(iri);
     }
 

src/main/java/com/networknt/schema/Validator.java

@@ -16,7 +16,7 @@
 
 package com.networknt.schema;
 
-import com.fasterxml.jackson.databind.JsonNode;
+import tools.jackson.databind.JsonNode;
 import com.networknt.schema.path.NodePath;
 import com.networknt.schema.walk.Walker;
 

src/main/java/com/networknt/schema/annotation/Annotations.java

@@ -20,7 +20,7 @@
 import java.util.List;
 import java.util.Map;
 
-import com.fasterxml.jackson.core.JsonProcessingException;
+import tools.jackson.core.JacksonException;
 import com.networknt.schema.path.NodePath;
 import com.networknt.schema.serialization.JsonMapperFactory;
 
@@ -92,7 +92,7 @@ public static String format(Map<NodePath, List<Annotation>> annotations) {
 
             try {
                 return JsonMapperFactory.getInstance().writerWithDefaultPrettyPrinter().writeValueAsString(results);
-            } catch (JsonProcessingException e) {
+            } catch (JacksonException e) {
                 return "";
             }
         }

src/main/java/com/networknt/schema/dialect/AbstractDialectRegistry.java

@@ -18,7 +18,7 @@
 import java.util.Map;
 import java.util.Map.Entry;
 
-import com.fasterxml.jackson.databind.JsonNode;
+import tools.jackson.databind.JsonNode;
 import com.networknt.schema.Error;
 import com.networknt.schema.InvalidSchemaException;
 import com.networknt.schema.Schema;