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

Remove Debug Vulnerability #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

briveramelo
Copy link

upgrading debug to 2.6.9 to eliminate the RegExp DOS low severity vulnerability per https://www.npmjs.com/advisories/534

@briveramelo
Copy link
Author

@may215 very simple change. Would love to hear your thoughts and clear this up

@briveramelo
Copy link
Author

briveramelo commented Mar 14, 2021

checking back on on this @may215

@briveramelo
Copy link
Author

@may215 @crobinson42 Can you please run npm update on this project to update the dependencies with vulnerabilities and publish an update? There is a certain irony to using a package about protection when it is the only source of known vulnerabilities in a project.

This is a simple fix.

@crobinson42
Copy link
Contributor

@briveramelo why are you tagging me in your comment? Quit bothersome.

@briveramelo
Copy link
Author

Are you not a contributor of this project? I imagine you are capable of doing what I've asked

@briveramelo
Copy link
Author

@crobinson42 The other reason, of course, is that it has been almost 3 years since I've requested this update, and now there is a 'high' level vulnerability in this dependency. Ultimately, I aim for a vulnerability-free project, and this is the one outstanding dependency. The fix is simple, but there has been no response.

I'm tagging you so this gets attention.

Accept the merge request.

@briveramelo
Copy link
Author

@crobinson42
Copy link
Contributor

@briveramelo You must not understand what a contributor and npm package owner are. I'll educate you:

Github Repository Contributor

A user who has made a change to a Github repo, ie: PR that is merged into the repo.

NPM Package Owner

A user(s) who has control or ownership of the NPM package to publish new package versions to the NPM repository.


You obviously found my name in the commit history and if you look at it with a little more diligence other than a lazy shotgun approach you would see I only suggested a Slack badge be added to the README.md.

I hope this explanation helps you be less annoying in the future, spread the word.

image

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.

2 participants