Do not pass "true" to getArraySchema to avoid StackOverflow on recursive models

Yaytay · Yaytay · commit fa0c25973db0 · 2025-10-24T09:01:31.000+01:00
diff --git a/modules/swagger-core/src/main/java/io/swagger/v3/core/jackson/ModelResolver.java b/modules/swagger-core/src/main/java/io/swagger/v3/core/jackson/ModelResolver.java
@@ -828,7 +828,7 @@ public Schema resolve(AnnotatedType annotatedType, ModelConverterContext context
                     if (reResolvedProperty.isPresent()) {
                         property = reResolvedProperty.get();
                     }
-                    reResolvedProperty = AnnotationsUtils.getArraySchema(ctxArraySchema, annotatedType.getComponents(), null, openapi31, property, true);
+                    reResolvedProperty = AnnotationsUtils.getArraySchema(ctxArraySchema, annotatedType.getComponents(), null, openapi31, property);
                     if (reResolvedProperty.isPresent()) {
                         property = reResolvedProperty.get();
                     }

Original file line number	Diff line number	Diff line change
`@@ -828,7 +828,7 @@ public Schema resolve(AnnotatedType annotatedType, ModelConverterContext context`
`828`	`828`	`if (reResolvedProperty.isPresent()) {`
`829`	`829`	`property = reResolvedProperty.get();`
`830`	`830`	`}`
`831`		`- reResolvedProperty = AnnotationsUtils.getArraySchema(ctxArraySchema, annotatedType.getComponents(), null, openapi31, property, true);`
	`831`	`+ reResolvedProperty = AnnotationsUtils.getArraySchema(ctxArraySchema, annotatedType.getComponents(), null, openapi31, property);`
`832`	`832`	`if (reResolvedProperty.isPresent()) {`
`833`	`833`	`property = reResolvedProperty.get();`
`834`	`834`	`}`