Mermade / oas-kit

Convert Swagger 2.0 definitions to OpenAPI 3.0 and resolve/validate/lint
https://mermade.org.uk/openapi-converter
BSD 3-Clause "New" or "Revised" License
696 stars 129 forks source link

Give an option to mark not required fields as nullable #635

Open gyz0072004 opened 1 year ago

gyz0072004 commented 1 year ago

Checklist

Detailed Description

Our API would return NULL value for optional fields in swagger definition. It's acceptable in OpenAPI 2.0. However, because the transformed OpenAPI 3.0 definition doesn't mark the field as nullable, the field becomes not nullable. We would like the toolkit to support auto marking the fields as nullable

Other stuff

swagger2openapi: 7.0.8

MikeRalphson commented 1 year ago

Could you expand on why this behaviour was acceptable in OpenAPI 2.0 (it doesn't seem obvious to me). Optional and nullable mean different things here. My gut feeling is this isn't something I'd be likely to add, but please try and convince me!