Skip to content

[8.x] [OAS] Fix handling of `schema.nullable` to path and query params (#197046)#197801

Merged
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-197046
Oct 25, 2024
Merged

[8.x] [OAS] Fix handling of `schema.nullable` to path and query params (#197046)#197801
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-197046

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…stic#197046)

## Summary

Fixes handling of `schema.nullable(schema.object({..}))` to params and
query inputs. [Example in the
wild](https://github.com/jloleysens/kibana/blob/83e76cb4d854a3c3f9ffdaad8c6ee29d66d56710/x-pack/plugins/reporting/server/routes/common/generate/request_handler.ts#L33).

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit d3569f6)
@kibanamachine kibanamachine requested a review from a team as a code owner October 25, 2024 11:16
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Oct 25, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) October 25, 2024 11:16
@kibanamachine kibanamachine merged commit 917fb71 into elastic:8.x Oct 25, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @jloleysens

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants