Bump jackson.version from 2.18.3 to 2.19.0 (#534)

* Bump jackson.version from 2.18.3 to 2.19.0

Bumps `jackson.version` from 2.18.3 to 2.19.0.

Updates `com.fasterxml.jackson.core:jackson-annotations` from 2.18.3 to 2.19.0
- [Commits](https://github.com/FasterXML/jackson/commits)

Updates `com.fasterxml.jackson.core:jackson-core` from 2.18.3 to 2.19.0
- [Commits](FasterXML/jackson-core@jackson-core-2.18.3...jackson-core-2.19.0)

Updates `com.fasterxml.jackson.core:jackson-databind` from 2.18.3 to 2.19.0
- [Commits](https://github.com/FasterXML/jackson/commits)

Updates `com.fasterxml.jackson.datatype:jackson-datatype-guava` from 2.18.3 to 2.19.0
- [Commits](FasterXML/jackson-datatypes-collections@jackson-datatypes-collections-2.18.3...jackson-datatypes-collections-2.19.0)

Updates `com.fasterxml.jackson.module:jackson-module-guice7` from 2.18.3 to 2.19.0
- [Commits](FasterXML/jackson-modules-base@jackson-modules-base-2.18.3...jackson-modules-base-2.19.0)

Updates `com.fasterxml.jackson.datatype:jackson-datatype-jdk8` from 2.18.3 to 2.19.0

Updates `com.fasterxml.jackson.datatype:jackson-datatype-jsr310` from 2.18.3 to 2.19.0

Updates `com.fasterxml.jackson.module:jackson-module-afterburner` from 2.18.3 to 2.19.0
- [Commits](FasterXML/jackson-modules-base@jackson-modules-base-2.18.3...jackson-modules-base-2.19.0)

---
updated-dependencies:
- dependency-name: com.fasterxml.jackson.core:jackson-annotations
  dependency-version: 2.19.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.fasterxml.jackson.core:jackson-core
  dependency-version: 2.19.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.fasterxml.jackson.core:jackson-databind
  dependency-version: 2.19.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.fasterxml.jackson.datatype:jackson-datatype-guava
  dependency-version: 2.19.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.fasterxml.jackson.module:jackson-module-guice7
  dependency-version: 2.19.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.fasterxml.jackson.datatype:jackson-datatype-jdk8
  dependency-version: 2.19.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.fasterxml.jackson.datatype:jackson-datatype-jsr310
  dependency-version: 2.19.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.fasterxml.jackson.module:jackson-module-afterburner
  dependency-version: 2.19.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* use properties instead of fields

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Brandon Arp <[email protected]>

Author: dependabot[bot]

pom.xml

@@ -81,7 +81,7 @@
     <fastutil.version>8.5.15</fastutil.version>
     <guava.version>33.4.8-jre</guava.version>
     <guice.version>7.0.0</guice.version>
-    <jackson.version>2.18.3</jackson.version>
+    <jackson.version>2.19.0</jackson.version>
     <javassist.version>3.30.2-GA</javassist.version>
     <javassist.maven.core.version>0.2.9</javassist.maven.core.version>
     <jsr305.version>3.0.2</jsr305.version>

src/main/java/com/arpnetworking/configuration/jackson/JsonNodeMergingSource.java

@@ -24,7 +24,6 @@
 import com.google.common.collect.Lists;
 import net.sf.oval.constraint.NotNull;
 
-import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
@@ -75,9 +74,7 @@ private static JsonNode merge(final JsonNode target, final JsonNode source) {
             // the target node. Any unmatched fields from the source node are
             // simply added to the target node; this requires a deep copy
             // since subsequent merges may modify it.
-            final Iterator<Map.Entry<String, JsonNode>> iterator = source.fields();
-            while (iterator.hasNext()) {
-                final Map.Entry<String, JsonNode> sourceFieldEntry = iterator.next();
+            for (final Map.Entry<String, JsonNode> sourceFieldEntry : source.properties()) {
                 final JsonNode targetFieldValue = target.get(sourceFieldEntry.getKey());
                 if (targetFieldValue != null) {
                     // Recursively merge the source field value into the target