jaxrs-analyzer icon indicating copy to clipboard operation
jaxrs-analyzer copied to clipboard

allow host to be omitted in Swagger back-end

Open abstratt opened this issue 7 years ago • 5 comments

In Swagger, the host is optional. If I want to generate a Swagger spec that can relative to the host serving it, you just omit the host property. Also, host is optional. Right now, if a deployment domain is not specified, it defaults to "" in SwaggerBackend, which is an invalid valid for the property. It should just be omitted instead,

abstratt avatar Jul 26 '17 18:07 abstratt

Similar/same as #42.

abstratt avatar Jul 26 '17 18:07 abstratt

Yes, that should do the trick... Would you like to help out with this feature and create a PR? :-)

sdaschner avatar Jul 27 '17 19:07 sdaschner

Also, should alliw for explicit API nsme and version. Currently, it uses the maven srtifact id and version.

abstratt avatar Jul 29 '17 21:07 abstratt

I have been doing this work as a contractor for Genologics/Illumina. Changes are in forks under their organization. Take a look and let me know if they seem good, and I can create a pull-request.

abstratt avatar Jul 29 '17 22:07 abstratt

Sorry for the late response. Yes looks good to me, thanks!

sdaschner avatar Nov 18 '17 20:11 sdaschner