Add JsonFactory.setStreamReadConstraints() (#964)

release-notes/VERSION-2.x

@@ -14,6 +14,10 @@ JSON library.
 === Releases ===
 ------------------------------------------------------------------------
 
+Not yet released
+
+#962: Offer a way to directly set `StreamReadConstraints` via `JsonFactory` (not just Builder)
+
 2.15.0-rc1 (18-Mar-2023)
 
 #827: Add numeric value size limits via `StreamReadConstraints` (fixes

src/main/java/com/fasterxml/jackson/core/JsonFactory.java

@@ -7,6 +7,7 @@
 import java.io.*;
 import java.lang.ref.SoftReference;
 import java.net.URL;
+import java.util.Objects;
 
 import com.fasterxml.jackson.core.format.InputAccessor;
 import com.fasterxml.jackson.core.format.MatchStrength;
@@ -276,7 +277,7 @@ public static int collectDefaults() {
      *
      * @since 2.15
      */
-    final protected StreamReadConstraints _streamReadConstraints;
+    protected StreamReadConstraints _streamReadConstraints;
 
     /**
      * Optional helper object that may decorate input sources, to do
@@ -778,6 +779,26 @@ public StreamReadConstraints streamReadConstraints() {
         return _streamReadConstraints;
     }
 
+    /**
+     * Method for overriding {@link StreamReadConstraints} defined for
+     * this factory.
+     *<p>
+     * NOTE: the preferred way to set constraints is by using
+     * {@link JsonFactoryBuilder#streamReadConstraints}: this method is only
+     * provided to support older non-builder-based construction.
+     * In Jackson 3.x this method will not be available.
+     *
+     * @param src Constraints
+     *
+     * @return This factory instance (to allow call chaining)
+     *
+     * @since 2.15
+     */
+    public JsonFactory setStreamReadConstraints(StreamReadConstraints src) {
+        _streamReadConstraints = Objects.requireNonNull(src);
+        return this;
+    }
+
     /*
     /**********************************************************
     /* Configuration, parser configuration

src/test/java/com/fasterxml/jackson/core/constraints/DeeplyNestedContentReadTest.java

@@ -37,6 +37,33 @@ private void _testDeepNesting(JsonParser p) throws Exception
         }
     }
 
+    public void testLegacyConstraintSettingTest() throws Exception
+    {
+        final int LOWER_MAX = 40;
+
+        final String DOC = createDeepNestedDoc(LOWER_MAX + 10);
+        JsonFactory f = new JsonFactory();
+        f.setStreamReadConstraints(StreamReadConstraints.builder()
+                .maxNestingDepth(LOWER_MAX).build());
+        for (int mode : ALL_STREAMING_MODES) {
+            try (JsonParser p = createParser(f, mode, DOC)) {
+                _testLegacyConstraintSettingTest(p, LOWER_MAX);
+            }
+        }
+    }
+
+    private void _testLegacyConstraintSettingTest(JsonParser p, int maxNesting) throws Exception
+    {
+        try {
+            while (p.nextToken() != null) { }
+            fail("expected StreamConstraintsException");
+        } catch (StreamConstraintsException e) {
+            assertEquals("Depth ("+(maxNesting+1)
+                    +") exceeds the maximum allowed nesting depth ("+maxNesting+")", e.getMessage());
+        }
+    }
+
+
     private String createDeepNestedDoc(final int depth) {
         StringBuilder sb = new StringBuilder();
         sb.append("[");