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

Upgrading lodash to the latest version #119

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jayakrishnagopal
Copy link

The current implementation of _.set in lodash ignores prototype to fix the prototype pollution issue mention by Snyk here - https://snyk.io/test/npm/lodash/4.17.15

I have implemented a simple set function to overcome this issue and help dependent projects move forward to the latest version of lodash.

@realdah
Copy link

realdah commented Feb 15, 2021

This is much required to be able to use mock-knex with latest lodash, please can owner review and approve this PR?

@jbrumwell jbrumwell closed this May 15, 2021
@jbrumwell jbrumwell reopened this May 15, 2021
@darrylivan
Copy link

Really appreciate this package!

I ran into this same problem. I would love to see this get into the latest version so we can move up lodash.

@unimaginativeGitHub
Copy link

Also hit this one.

Mock-knex is great! Even better with this shiny new set function 😉

@LogansUA
Copy link

LogansUA commented Aug 1, 2022

Hey, @jbrumwell any thoughts on this one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants