-
Notifications
You must be signed in to change notification settings - Fork 112
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
Found vulnerabilities after 'npm install' #68
Comments
@dragonBall561 There is no reason to be alarmed. NPM checks dependencies of all packages. The vulnerabilities are in components that Dopamine uses, and in sub-components of those components. Trying to fix those, is madness. |
That article (an opinion piece) basically says that, if a vulnerability was found, it does exist and the author won’t care anyway. Among the several reasons why I, personally, think that the move towards Electron was a bad idea, this is one of them. Yes, you will fetch dependencies with security problems, and no, this is not something you should ignore. I would prefer software that I install and use to have no known security problems, to be honest. |
after running npm install I get:
But Dopamine Runs Fine.
Are the vulnerabilities bad?
The text was updated successfully, but these errors were encountered: