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

Added openapi 3.1 #1951

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Added openapi 3.1 #1951

wants to merge 1 commit into from

Conversation

carloscbl
Copy link

Fixes #1396 .

connexion 3 was supposed to support openapi 3.1

Changes proposed in this pull request:

  • This PR is incomplete, but it allows you to load an 3.1 openapi and the later you can use your custom validators.
  • Without this changes you cannot even try to load openapi 3.1
  • Maybe maintainers want to upgrade default validators from draf4 to at least draft 7 or latest or just leave as it is and you can just extend the validator
  • Or make validators change if you are using openapi 3.1
  • Also tests are missing for openapi 3.1

@sameerhussain
Copy link

Hi,
Woud you please provide any updates in this regards @carloscbl ?

If you need any help, I am ready to help here to add more code to get this merged.

@carloscbl carloscbl changed the title WIP Added openapi 3.1 Added openapi 3.1 Jan 7, 2025
@carloscbl
Copy link
Author

Hi, Woud you please provide any updates in this regards @carloscbl ?

If you need any help, I am ready to help here to add more code to get this merged.

I have this working for some time in a custom version. I want to clarify:

This gives support to openapi 3.1 connexion works perfectly with it and validates the schema, requests and responses.

But is not a 100% openapi 3.1 validation some other parts of the code reference some kind of validation that is using older drafts.

On my day to day for our extensive uses cases we found no problems running it.

So imho is ready to merge.

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.

OpenAPI Specification 3.1
2 participants