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

README.md

@@ -411,17 +411,17 @@ JsonLocation minLengthInstanceNodeTokenLocation = JsonNodes.tokenStreamLocationO
 JsonLocation minLengthSchemaNodeTokenLocation = JsonNodes.tokenStreamLocationOf(minLength.getSchemaNode());
 
 assertEquals("format", format.getKeyword());
-assertEquals("date", format.getSchemaNode().asText());
+assertEquals("date", format.getSchemaNode().asString());
 assertEquals(5, formatSchemaNodeTokenLocation.getLineNr());
 assertEquals(17, formatSchemaNodeTokenLocation.getColumnNr());
-assertEquals("1", format.getInstanceNode().asText());
+assertEquals("1", format.getInstanceNode().asString());
 assertEquals(2, formatInstanceNodeTokenLocation.getLineNr());
 assertEquals(16, formatInstanceNodeTokenLocation.getColumnNr());
 assertEquals("minLength", minLength.getKeyword());
-assertEquals("6", minLength.getSchemaNode().asText());
+assertEquals("6", minLength.getSchemaNode().asString());
 assertEquals(6, minLengthSchemaNodeTokenLocation.getLineNr());
 assertEquals(20, minLengthSchemaNodeTokenLocation.getColumnNr());
-assertEquals("1", minLength.getInstanceNode().asText());
+assertEquals("1", minLength.getInstanceNode().asString());
 assertEquals(2, minLengthInstanceNodeTokenLocation.getLineNr());
 assertEquals(16, minLengthInstanceNodeTokenLocation.getColumnNr());
 assertEquals(16, minLengthInstanceNodeTokenLocation.getColumnNr());

doc/custom-dialect.md

@@ -35,7 +35,7 @@ public class EqualsValidator extends BaseKeywordValidator {
     @Override
     public void validate(ExecutionContext executionContext, JsonNode node, JsonNode rootNode,
             NodePath instanceLocation) {
-        if (!node.asText().equals(value)) {
+        if (!node.asString().equals(value)) {
             executionContext.addError(error().message("must be equal to ''{0}''")
                             .arguments(value)
                             .instanceLocation(instanceLocation).instanceNode(node).evaluationPath(executionContext.getEvaluationPath()).build());

doc/walkers.md

@@ -95,14 +95,14 @@ assertEquals("", propertyKeywords.get(0).getInstanceLocation().toString());
 assertEquals("/properties",
         propertyKeywords.get(0).getEvaluationPath().append(propertyKeywords.get(0).getKeyword()).toString());
 assertEquals("/tags/0", propertyKeywords.get(1).getInstanceLocation().toString());
-assertEquals("image", propertyKeywords.get(1).getInstanceNode().get("name").asText());
+assertEquals("image", propertyKeywords.get(1).getInstanceNode().get("name").asString());
 assertEquals("/properties/tags/items/$ref", propertyKeywords.get(1).getEvaluationPath().toString());
 assertEquals("/properties/tags/items/$ref/properties",
         propertyKeywords.get(1).getEvaluationPath().append(propertyKeywords.get(1).getKeyword()).toString());
 assertEquals("/tags/1", propertyKeywords.get(2).getInstanceLocation().toString());
 assertEquals("/properties/tags/items/$ref/properties",
         propertyKeywords.get(2).getEvaluationPath().append(propertyKeywords.get(2).getKeyword()).toString());
-assertEquals("link", propertyKeywords.get(2).getInstanceNode().get("name").asText());
+assertEquals("link", propertyKeywords.get(2).getInstanceNode().get("name").asString());
 ```
 
 ### Applying Defaults

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</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>25.0.0</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>
@@ -143,7 +143,16 @@
       <!-- Used to validate ECMA 262 regular expressions -->
       <!-- Approximately 50 MB in dependencies -->
       <groupId>org.graalvm.js</groupId>
-      <artifactId>js</artifactId>
+      <artifactId>js-language</artifactId>
+      <version>${version.graaljs}</version>
+      <optional>true</optional>
+    </dependency>
+
+    <dependency>
+      <!-- Used to validate ECMA 262 regular expressions -->
+      <!-- Approximately 50 MB in dependencies -->
+      <groupId>org.graalvm.truffle</groupId>
+      <artifactId>truffle-runtime</artifactId>
       <version>${version.graaljs}</version>
       <optional>true</optional>
     </dependency>
@@ -459,7 +468,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;
@@ -107,11 +107,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 {
@@ -519,7 +519,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;
@@ -537,18 +537,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);
     }