-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Management API OpenAPI document is not valid #405
Comments
Thanks for reporting @duuri-mollinen Interesting, we do lint with https://github.com/stoplightio/spectral lavinmq/build/build_docs_in_ci Lines 5 to 9 in 9eab401
Maybe Spectral can't catch errors like these? Do you know if it is possible to lint with swagger-ui (or some other tool from the swagger project)? We would like to be able to run it command line. The JSON you can download is provided by redoc-cli as part of the generated docs Lines 35 to 36 in 9eab401
but I checked the first error here and indeed lavinmq/openapi/schemas/nodes.yaml Lines 122 to 123 in 9eab401
|
That at least claims do the trick https://github.com/char0n/swagger-editor-validate I'm not sure about your tool chain, if that's directly suitable or not |
Thanks for that tip, I see it runs Swagger UI using Chrome headless via Puppeteer 😮 Not sure we want to go down that route, I'll need to do some research |
Tried https://github.com/APIDevTools/swagger-cli but it does not catch these errors
Tried with too APIDevTools/swagger-cli#78 , didn't help |
This seems to be something at least (no line numbers given put those aren't really relevant anyway as the source is YAML), still an open question is how to produce the JSON in CI (and put it online?)
|
Looks like it is https://github.com/swagger-api/validator-badge that's powering https://validator.swagger.io/, possible to run using Docker |
https://docs.lavinmq.com/http-api.html -> download -> https://editor.swagger.io ->
The text was updated successfully, but these errors were encountered: