You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
At the moment it seems the v3.Parameter's explode field is just a boolean. Meaning if its not set in the OpenAPI document it will default to the boolean's zero value of false. But in the case of query parameters for example, the default value should be true, and because the type is boolean (as opposed to *boolean) I can't tell if it is unset in the document and I should default handling of explode to true.
The text was updated successfully, but these errors were encountered:
Just adding to this the v3.Encoding explode field should be handle the same (as a pointer) as it can default to true as well if style == form and its not provided
…onal property using pointer. #26@TristanSpeakEasy notes: "At the moment it seems the v3.Parameter's explode field is just a boolean. Meaning if its not set in the OpenAPI document it will default to the boolean's zero value of false. But in the case of query parameters for example, the default value should be true, and because the type is boolean (as opposed to *boolean) I can't tell if it is unset in the document and I should default handling of explode to true."
Both `explode` properties have been moved to optional properties using pointers.
Signed-off-by: Dave Shanley <dave@quobix.com>
At the moment it seems the
v3.Parameter
'sexplode
field is just aboolean
. Meaning if its not set in the OpenAPI document it will default to the boolean's zero value offalse
. But in the case of query parameters for example, the default value should betrue
, and because the type isboolean
(as opposed to*boolean
) I can't tell if it is unset in the document and I should default handling of explode totrue
.The text was updated successfully, but these errors were encountered: