Skip to content
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.

Sign up for GitHub

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

Jump to bottom

OpenAPI 3.1 broken reference to child class with oneOf reference #4634

Closed
GooDer opened this issue Mar 20, 2024 · 1 comment
Closed

OpenAPI 3.1 broken reference to child class with oneOf reference #4634

GooDer opened this issue Mar 20, 2024 · 1 comment
Assignees
@micryc

Comments

@GooDer
Copy link
Contributor

GooDer commented Mar 20, 2024

Hi, I encountered another bug related to new OpenAPI 3.1 version.

When we have some object which's attribute is another object which is abstract and has at lease one implementation class it will not be rendered and error is displayed in swagger UI.

Version 3.0 output:
image

Version 3.1 output:
image

I found out that it is caused again by change in element class from ComposedSchema to JsonSchema.

I will try to provide fix also tomorrow.

Minimalistic project to reproduce:
swagger-core-bug.zip
@quovide quovide mentioned this issue Apr 16, 2024
Open
@micryc micryc closed this as completed in 2c1d0e3 May 13, 2024
micryc added a commit that referenced this issue May 13, 2024
@micryc
Merge pull request #4637 from GooDer/openApi-3.1-ComposedSchema-fix
8660f35 
Fix #4634: unite processing of allOf, anyOf, oneOf relation so it wil…
@micryc micryc mentioned this issue May 13, 2024
Merged
@micryc micryc self-assigned this May 13, 2024
@micryc
Copy link
Contributor

micryc commented May 13, 2024

Thank you @GooDer for reporting issue and providing fix PR. Closing ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@micryc micryc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@GooDer @micryc