swagger-api / swagger-parser

Swagger Spec to Java POJOs
http://swagger.io
Apache License 2.0
773 stars 525 forks source link

SwaggerParse Results need to refactor #2100

Open sagartib opened 1 month ago

sagartib commented 1 month ago

SwaggerParseResults.. should show what are the warnings and errors, by some prefix or separate list like in ParseRoot..

Please help to fix.

sagartib commented 1 month ago

SwaggerParseResult giving merged results of Warnings and Error. there is no way can clearly separate from this list.

public SwaggerParseResult deserialize(JsonNode rootNode, String path) { SwaggerParseResult result = new SwaggerParseResult(); ParseResult rootParse = new ParseResult(); OpenAPI api = this.parseRoot(rootNode, rootParse, path); result.setOpenAPI(api); result.setMessages(rootParse.getMessages()); return result; }

However required results available in OpenAPIDeserializer- ParseResult,, which is protected.

if SwaggerParseResult output gives or other way to get warnings and errors separately, that would be helpful.. People can use them whether the swagger is valid or not.

or

allow them ParseOption to disable warnings and just return errors.