Skip to content
GitLab
Closed
Issue created by Administrator@rootContributor0 of 3 checklist items completed0/3 checklist items

[BUG] StackOverflowError in DefaultCodegen.addProperties when a schemas with same name are referenced directly and indirectly

Created by: ivo500

Bug Report Checklist

  • [Y ] Have you provided a full/minimal spec to reproduce the issue?
  • [ Y] Have you validated the input using an OpenAPI validator (example)?
  • What's the version of OpenAPI Generator used? 4.2.1
  • [Y ] Have you search for related issues/PRs?
  • What's the actual output vs expected output? java.lang.StackOverflowError, expected Sring generated project
  • [Optional] Bounty to sponsor the fix (example)
Description

When trying to generate spring project from a complex .yaml file setup a java.lang.StackOverflowError is thrown.

The setup is the following:

Base.yaml contains a definition of Test Blue.yaml contains a definition of Test which inherits with Allof from Base.yaml->Test Green.yaml also contains a definition of Test which inherits with Allof from Base.yaml->Test and also it has a reference to Blue.yaml->Test

When generation is started for Green.yaml , DefaultCodegen.addProperties causes stack overflow.

Generator first loads the Base.yaml->Test reference in its cache. The problem happens when the generator starts processing the Blue.yaml->Test reference in Green.yaml. It loads the Blue.yaml->Test but wrongly overrides the Base.yaml->Test in its cache. Blue.yaml->Test is inheriting from Base.yaml->Test but now after the override Blue.yaml->Test inherits from itself and becomes corrupt!!!!!!!!!. After this when DefaultCodegen.addProperties starts its work it goes into endless recursion trying to process the corrupted self referencing Blue.yaml->Test .

See the stack trace below:

[main] INFO o.o.codegen.DefaultGenerator - OpenAPI Generator: spring (server) [main] INFO o.o.codegen.DefaultGenerator - Generator 'spring' is considered stable. [main] INFO o.o.codegen.languages.SpringCodegen - ---------------------------------- [main] INFO o.o.c.languages.AbstractJavaCodegen - Environment variable JAVA_POST_PROCESS_FILE not defined so the Java code may not be properly formatted. To define it, try 'export JAVA_POST_PROCESS_FILE="/usr/local/bin/clang-format -i"' (Linux/Mac) [main] INFO o.o.c.languages.AbstractJavaCodegen - NOTE: To enable file post-processing, 'enablePostProcessFile' must be set to true (--enable-post-process-file for CLI). [main] INFO o.o.c.languages.AbstractJavaCodegen - Processing operation test [main] INFO o.o.c.languages.AbstractJavaCodegen - Processing operation inbox_storeTaskResponse Exception in thread "main" java.lang.StackOverflowError at org.apache.commons.lang3.StringUtils.isNotBlank(StringUtils.java:327) at org.openapitools.codegen.DefaultCodegen.addProperties(DefaultCodegen.java:2020) at org.openapitools.codegen.DefaultCodegen.addProperties(DefaultCodegen.java:2002) at org.openapitools.codegen.DefaultCodegen.addProperties(DefaultCodegen.java:2022) at org.openapitools.codegen.DefaultCodegen.addProperties(DefaultCodegen.java:2002) at org.openapitools.codegen.DefaultCodegen.addProperties(DefaultCodegen.java:2022) at org.openapitools.codegen.DefaultCodegen.addProperties(DefaultCodegen.java:2002) at org.openapitools.codegen.DefaultCodegen.addProperties(DefaultCodegen.java:2022) at org.openapitools.codegen.DefaultCodegen.addProperties(DefaultCodegen.java:2002) at org.openapitools.codegen.DefaultCodegen.addProperties(DefaultCodegen.java:2022) at org.openapitools.codegen.DefaultCodegen.addProperties(DefaultCodegen.java:2002)

openapi-generator version

4.2.1

OpenAPI declaration file content or url
Command line used for generation

openapi-generator generate -i task-test.yaml -g spring -o test

Steps to reproduce
Related issues/PRs
Suggest a fix

The problem seem to be that when loading a second ref .yaml file which contains schemas with the same name as a component from the other first ref.yaml file. The second component overrides the first component.