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

OpenAPI 3.1 support? #373

Open
mitar opened this issue May 25, 2021 · 11 comments · May be fixed by #1211
Open

OpenAPI 3.1 support? #373

mitar opened this issue May 25, 2021 · 11 comments · May be fixed by #1211
Labels
enhancement New feature or request upstream

Comments

@mitar
Copy link
Contributor

mitar commented May 25, 2021

OpenAPI 3.1 is great, brings full JSON Schema compatibility. It would be great if generator would support it.

@eli-kaplan
Copy link

Bumping this, OAS 3.1 was released approximately seven months ago now and brings a number of very useful features.

@sonu27
Copy link
Contributor

sonu27 commented Sep 14, 2021

This would be great

@jamietanna
Copy link
Collaborator

So right now you can use a document with an OpenAPI 3.1 version, but we don't yet support new stuff.

There's a great post about what we need to add at https://www.openapis.org/blog/2022/05/11/openapi-v3-1-resources-for-tooling-developers

Is there anything we want to count as out of scope? I'd be tempted to look at whether I can help with the effort to add support

@jamietanna
Copy link
Collaborator

Related upstream issue that we'd need to wait for before we can support everything getkin/kin-openapi#230

@jamietanna
Copy link
Collaborator

jamietanna commented Oct 19, 2022

Per getkin/kin-openapi#230 (comment), https://github.com/pb33f/libopenapi could also be a good option, but would lead to significant rework - may be worth it, especially as kin-openapi's not yet gotten around to it

Edit: maybe when it's a little more stable?

@amery
Copy link
Contributor

amery commented Oct 26, 2022

If you end up redesigning oapi-codegen using libopenapi, could you please consider allowing generating code for other languages too? I'm sure people here would be interested in using the same awesome tool to generate non-go client code

@jamietanna
Copy link
Collaborator

Just to cover this, there's some discussion between @deepmap-marcinr and I about this, and we're considering - for now - to convert an OpenAPI 3.1 spec to an OpenAPI 3.0 spec (via libopenapi) to continue to rely on kin-openapi, which will allow for OpenAPI 3.1 specs to continue to work, but without some of the benefits.

We'd prefer to keep on kin-openapi for the meantime, but also if there's no expected update to 3.1 (and folks do really want 3.1) we can look at shifting to use libopenapi, but it'll require a slightly more significant set of changes than we're willing to do right now, especially as Marcin is continuing to think about what a v2 could look like

@jamietanna
Copy link
Collaborator

For folks looking for OpenAPI 3.1 support, what features are you hoping for? We are looking at what initial changes may be required (supporting multiple types in a schema), but then wondering what additional things could be added to the backlog, like webhook support

@mgabeler-lee-6rs
Copy link
Contributor

For folks looking for OpenAPI 3.1 support, what features are you hoping for?

For my team's use cases, you nailed the two key items on the head -- type as an array, and webhooks, in that order of importance.

The type array could definitely be translated to an OAS 3.0 spec pretty easily, by converting it to a oneOf or anyOf, though nullability has some minor challenges in some cases.

@wjkoh
Copy link

wjkoh commented Feb 28, 2024

The fully implemented version of OpenAPI 3.1 can be found at https://github.com/pb33f/libopenapi, while kin-openapi is still in the process of implementing it. Would switching to the former accelerate the support for OAS 3.1?

@jamietanna
Copy link
Collaborator

So using libopenapi is proposed in #1388 and is a good option to support OpenAPI 3.1.

It's not something I want us to switch over to as-is, as it's a significant breaking change for folks (for those using this as a library, and some behaviour changes may occur) so would assume this would be a side-by-side thing for some time before maybe switching over.

Additionally, we also need to make sure the middleware used by the project is also https://github.com/pb33f/libopenapi-validator/ instead of using kin-openapi (as they'll differ in what's supported)

I'd welcome folks trying out #1388 with their repos, making sure that it works correctly with your services, etc, and we'll hopefully get it merged before long, but I want to make sure that we set ourselves up for it well due to i.e. needing to make sure that updates to internal code for generating templates also applies to the libopenapi side of things.

It's very much something I want to get done, just also trying to balance it with several other Open Source projects I maintain (all in my free time) as well as trying to keep on top of the incoming issues and PRs!

@jamietanna jamietanna added enhancement New feature or request upstream labels May 3, 2024
jamietanna added a commit that referenced this issue May 5, 2024
As noted in #373 we don't yet have support, and this should make it
clearer to consumers of that.
jamietanna added a commit that referenced this issue May 5, 2024
As noted in #373 we don't yet have support, and this should make it
clearer to consumers of that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request upstream
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants