Fix OpenApiEncoding explode property serialization - #2508
Conversation
f6c558d to
c14dd24
Compare
Vincent Biret (baywet)
left a comment
There was a problem hiding this comment.
Thanks for the contribution!
|
David Brodski (@kammerjaeger) can you please also sign the CLA? (see the bot comment) |
|
@microsoft-github-policy-service agree company="AMD" |
|
Thanks for the fast review. How often do you release new versions? |
|
We make releases as needed, as long as it's possible because this is a maintenance branch. Once we release main, I'll re-run it manually, please ping me if I forget :) |
|
David Brodski (@kammerjaeger) I had to trigger the release manually to circumvent the the issue, I'm waiting for somebody to approve it #2513 |
Fix: Explode serialization is wrong when style is "form"
This fixes the same problem as #544 but for OpenApiEncoding.
The spec describes different behavior depending on if it is set or not (see: https://spec.openapis.org/oas/v3.1.0.html#fixed-fields-12) and also different defaults depending on the style value.
This also needs to be cherry picked to main, I just need it for v1 but will create a pr to main with the change if this pr is fine.
Please let me know what you think.