Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix for stripUnknown not working for nested object on invalid object #2518

Merged

Conversation

aperwenis
Copy link
Contributor

@aperwenis aperwenis commented Nov 26, 2020

Fixes #2479
Fixes #2459

@Unibozu
Copy link

Unibozu commented Jan 20, 2021

Hey

Just wanted to say I am also blocked by this issue on a project that is supposed to be released in a few days (we've fixed it with a hefty amount of JS to re-structure the objects but it would be amazing to get Joi to do it for us since it's fully aware of the schema).

This PR has been waiting for a while and seems to be fixing two issues, is it possible to review it? perhaps have I missed something that may block merging it?

Thanks everyone

@hueniverse hueniverse self-assigned this Feb 8, 2021
@hueniverse hueniverse added the bug Bug or defect label Feb 8, 2021
@hueniverse hueniverse added this to the 17.4.0 milestone Feb 8, 2021
@hueniverse hueniverse merged commit 8a6349c into hapijs:master Feb 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug or defect
Projects
None yet
3 participants