-
Notifications
You must be signed in to change notification settings - Fork 944
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
[Backport 1.3] [1.x backport] Bump joi
to v14 to avoid the possibility of prototype poisoning in a nested dependency
#4345
Conversation
…e poisoning in a nested dependency (#4211) Backport PR #3952 Signed-off-by: Miki <[email protected]> Co-authored-by: Miki <[email protected]> (cherry picked from commit 4626066) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> # Conflicts: # CHANGELOG.md
Codecov Report
@@ Coverage Diff @@
## 1.3 #4345 +/- ##
==========================================
+ Coverage 67.45% 67.50% +0.04%
==========================================
Files 3044 3044
Lines 58692 58692
Branches 8902 8902
==========================================
+ Hits 39591 39619 +28
+ Misses 16947 16925 -22
+ Partials 2154 2148 -6
Flags with carried forward coverage won't be shown. Click here to find out more. |
I am not happy to see |
This is a major version bump - I didn't see any analysis about how/why this is safe to backport to the 1.x line - can we provide that here? |
Signed-off-by: Josh Romero <[email protected]>
here are the breaking changes for going from 13 to 14 hapijs/joi#1615
Based on this i think this is safe to backport to 1.3 |
Signed-off-by: Qingyang(Abby) Hu <[email protected]>
Backport 4626066 from #4211.