Properly handle empty responses in OpenAPI.

noboomu · Oct 22, 2018 · b7d5cf6 · b7d5cf6
1 parent 09a4f4c
commit b7d5cf6
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/src/main/java/io/sinistral/proteus/server/tools/oas/Reader.java b/src/main/java/io/sinistral/proteus/server/tools/oas/Reader.java
@@ -1348,12 +1348,14 @@ else if (classType.getRawClass().isAssignableFrom(CompletableFuture.class))
 		}
 		if (operation.getResponses() == null || operation.getResponses().isEmpty())
 		{
-			Content content = new Content();
-			MediaType mediaType = new MediaType();
-			AnnotationsUtils.applyTypes(classProduces == null ? new String[0] : classProduces.value(),
-										methodProduces == null ? new String[0] : methodProduces.value(), content, mediaType);
+			LOGGER.debug("responses are null or empty");
+
+			//Content content = new Content();
+			//MediaType mediaType = new MediaType();
+			//AnnotationsUtils.applyTypes(classProduces == null ? new String[0] : classProduces.value(),
+			//							methodProduces == null ? new String[0] : methodProduces.value(), content, mediaType);
 
-			ApiResponse apiResponseObject = new ApiResponse().description(DEFAULT_DESCRIPTION).content(content);
+			ApiResponse apiResponseObject = new ApiResponse().description(DEFAULT_DESCRIPTION);//.content(content);
 			operation.setResponses(new ApiResponses()._default(apiResponseObject));
 		}
 
@@ -1495,7 +1497,7 @@ private void setOperationObjectFromApiOperationAnnotation(
 				operation.setResponses(responses);
 			}
 			else
-			{
+			{ 
 				responses.forEach(operation.getResponses()::addApiResponse);
 			}
 		});