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 security vulnerabilities #343

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

Conversation

Berkmann18
Copy link

@Berkmann18 Berkmann18 commented Jun 5, 2019

Fix #312 and #331.

Note: some vulnerabilities couldn't be fixed here as they are located within vorpal-repl and load-plugins.

@superliuwr
Copy link

Hey @Berkmann18, appreciated your work! I had issues with the lodash vulnerability too and this PR will help.
Looks like it's broken the CI though. Would you mind having another look?

@Berkmann18
Copy link
Author

@superliuwr Yeah, the second commit did break it, I'll try to revert a part of it and see.

@orpheus
Copy link

orpheus commented Nov 7, 2019

Merge!

@Berkmann18
Copy link
Author

@dthree ?

@sweethuman
Copy link

@dthree has been inactive since 2018 as you can see...

@reiser
Copy link

reiser commented Feb 3, 2021

+1

@rediris
Copy link

rediris commented Mar 19, 2021

@dthree any hope of getting this merged? I see you've been active in github recently.

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.

Node Security Issue with lodash: 577 - Prototype Pollution
7 participants