-
-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[BUG] Unable to load complex definition with referenced schemas #16184
Comments
I tested with the latest master and it works for me:
Can you please give it a try? FYI. 7.0.0-beta was released 2 weeks ago. |
The beta does indeed work; thank you for pointing that out. I usually try to use what anyone would pick up. |
I switched the OpenAPI doc to version 3.0.3 as I was chasing a further issue and now it breaks again, even with latest. Key error appears to be this:
My definition uses "$ref" inside "additionalProperties", which is legitimate. The online tool masks that error and simply shows an empty model for the return values, e.g. of GET /definitionGroup |
ok, will investigate |
@wing328 - I put this specific iteration of the spec doc into its own branch so that I can keep on working on my PR; I'll leave this untouched: There are also these two siblings: |
Bug Report Checklist
Description
openapi-generator-cli generate -g csharp -i https://raw.githubusercontent.com/clemensv/spec-1/fix-schema2/message/schemas/xregistry_openapi_messagedefinition.json
The issue occurs with all generators and during the loading phase.
openapi-generator version
6.6.0
OpenAPI declaration file content or url
https://raw.githubusercontent.com/clemensv/spec-1/fix-schema2/message/schemas/xregistry_openapi_messagedefinition.json
Generation Details
openapi-generator-cli generate -g csharp -i https://raw.githubusercontent.com/clemensv/spec-1/fix-schema2/message/schemas/xregistry_openapi_messagedefinition.json
Steps to reproduce
see above
Output:
(truncated)
The text was updated successfully, but these errors were encountered: