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.

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

openapi3: fix abort logic of validation for schemas with anyOf, allOf, oneOf #892

Merged
merged 2 commits into from
Jan 22, 2024

Conversation

AmadeusK525
Copy link
Contributor

Description

Only abort further validation if xOf validations were successful AND the value is nil (this means that the xOf was nullable for a fact). Otherwise, continue validating against the parent schema

Related Issues

@micronull
Copy link
Contributor

micronull commented Jan 9, 2024

@AmadeusK525 @fenollp Checked with ourselves. Problem solved. Thank you!

Waiting for the release.

Copy link
Collaborator

@fenollp fenollp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! LGTM.
Please merge in changes profusion#1

@fenollp
Copy link
Collaborator

fenollp commented Jan 22, 2024

@AmadeusK525 Hey please merge profusion#1 :)

@fenollp fenollp merged commit c3d9748 into getkin:master Jan 22, 2024
7 checks passed
@AmadeusK525
Copy link
Contributor Author

Hey sorry @fenollp, missed your ping, but I included your changes in the commits, thanks! Good to know this issue was resolved! cc: @micronull

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🪲 No further validation in the presence of anyOf. Always true.
3 participants