search

sdaschner / jaxrs-analyzer

Creates REST documentation for JAX-RS projects
Apache License 2.0
319 stars 101 forks source link

allow host to be omitted in Swagger back-end #122

Open abstratt opened 7 years ago

abstratt commented 7 years ago

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 commented 7 years ago

Similar/same as #42.

sdaschner commented 7 years ago

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

abstratt commented 7 years ago

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

abstratt commented 7 years ago

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.

sdaschner commented 6 years ago

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