oapi-codegen

oapi-codegen copied to clipboard

fix: avoid stack overflow errors when using heavily recursive types

1

# Issue https://github.com/deepmap/oapi-codegen/issues/1373 Stack overflow error happens `merge_schemas.go` when `allOf` contains circular `$ref`. # Cause & How to fix https://github.com/deepmap/oapi-codegen/blob/1f53862bcc64573d3d0c4c105c71a8143e7b1816/pkg/codegen/merge_schemas.go#L87 `mergeOpenapiSchemas` is called recursively until all items in `allOf` are...

yoshikipom

As it doesn't actually do anything, as it wraps an existing server i.e. Chi. Closes #1558. --- Maybe this should be split into a separate option to add the `strict-server-models`?...

jamietanna

https://github.com/deepmap/oapi-codegen/issues/1558

jamietanna

(Not yet tested)

jamietanna

feat: provide a complex `overrides` configuration

1

For much more control over how `oapi-codegen` works - `overrides.spec` to override a JSON path in the OpenAPI specification - `overrides.schema` to override the derived `codegen.Schema` that `oapi-codegen` generates

jamietanna

i.e. `ComponentsSchemaPet` to make sure that everything is definitely not going to clash (as we've had before with i.e. `Client`)

jamietanna

To lead into folks being able to configure #1388 or the existing `kin-openapi`. By default, we'll have `kin-openapi`.

jamietanna

Warn when `x-deprecate-reason` is set but not `deprecated: true`

1

jamietanna

Embedded spec sometimes omits schemas from external refs, race-like

7

I have a couple OAS specs in my app. One of them loads a schema from another spec file, using something like this: ```yaml components: specs: MyFoo: $ref: '../otherdir/otherspec.yaml#/components/schemas/Foo' ```...

mgabeler-lee-6rs

https://github.com/deepmap/oapi-codegen/blob/master/configuration-schema.json#L65

jamietanna