- Fix stackover flow on OpenAPI generation

Author: maxtuzz

src/main/java/io/dinject/javalin/generator/Processor.java

@@ -73,11 +73,10 @@ private void writeControllerAdapter(Element controller) {
       try {
         ControllerRouteWriter writer = new ControllerRouteWriter(reader, ctx);
         writer.write();
-      } catch (Exception e) {
+      } catch (Throwable e) {
         e.printStackTrace();
-        ctx.logError(reader.getBeanType(), "Failed to write $route class");
+        ctx.logError(reader.getBeanType(), "Failed to write $route class " + e);
       }
     }
   }
-
 }

src/main/java/io/dinject/javalin/generator/openapi/SchemaDocBuilder.java

@@ -148,8 +148,10 @@ private Schema<?> buildObjectSchema(TypeMirror type) {
 
     Schema objectSchema = schemas.get(objectSchemaKey);
     if (objectSchema == null) {
-      objectSchema = createObjectSchema(type);
+      // Put first to resolve recursive stack overflow
+      objectSchema = new ObjectSchema();
       schemas.put(objectSchemaKey, objectSchema);
+      populateObjectSchema(type, objectSchema);
     }
 
     Schema ref = new Schema();
@@ -209,10 +211,7 @@ private String getObjectSchemaName(TypeMirror type) {
     return canonicalName;
   }
 
-  private ObjectSchema createObjectSchema(TypeMirror objectType) {
-
-    ObjectSchema objectSchema = new ObjectSchema();
-
+  private <T> void populateObjectSchema(TypeMirror objectType, Schema<T> objectSchema) {
     Element element = types.asElement(objectType);
     for (VariableElement field : allFields(element)) {
       Schema<?> propSchema = toSchema(field.asType());
@@ -223,7 +222,6 @@ private ObjectSchema createObjectSchema(TypeMirror objectType) {
       setFormatFromValidation(field, propSchema);
       objectSchema.addProperties(field.getSimpleName().toString(), propSchema);
     }
-    return objectSchema;
   }
 
   private void setFormatFromValidation(Element element, Schema<?> propSchema) {