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

"The ESLint extension will use 'node_modules/eslint' for validation" make this warning go away. #1023

Closed
nojvek opened this issue Jul 21, 2020 · 14 comments

Comments

@nojvek
Copy link

nojvek commented Jul 21, 2020

The ESLint extension will use 'node_modules/eslint' for validation, which is installed locally in 'panel'. If you trust this version of ESLint, press 'Allow', otherwise press 'Do Not Allow'. Press 'Cancel' to disable ESLint for this session.

I usually have multiple projects open at any time. This warning appears every single time and is quite annoying. Is there a setting to turn it off. Why does it block the user's flow?

@dbaeumer
Copy link
Member

@nojvek pls see #1012. It is basically a security fix.

Actually you should only be prompted once per workspace folder unless you press Cancel. Do you see the prompt more often.

Closing as dup of #1012

@nojvek
Copy link
Author

nojvek commented Jul 21, 2020 via email

@dbaeumer
Copy link
Member

@nojvek which button do you press on the dialog? And which version of the ESLint extension are you using. We had a bug that is fixed in 2.1.8.

@nojvek
Copy link
Author

nojvek commented Jul 23, 2020

I was on latest (2.1.8). But I downgraded to eslint without that annoying modal. Thank god vscode lets you choose the version.

@gonpeche
Copy link

I accidentally pressed "cancel" and now I cant get the ESLint format while typing...
Is there a way to pop-up again the original message and allow it?

@dbaeumer
Copy link
Member

Yes, run the command ESLint: Reset Library Decision

@nojvek
Copy link
Author

nojvek commented Jul 29, 2020

At this point I don’t know how much value that modal is adding. Reminds me of windows vista popups. So many that users just learn to ignore them. Why can’t the plugin just do an integrity check of the node_modules installed eslint.

Even if you show the modal, a user doesn’t really know what to do other than clicking a button. If you really care about security then go the extra mile and only alert when locally installed eslint doesn’t match a verified list of hashes.

@dbaeumer
Copy link
Member

The problem is that ESLint itself has a plugin system. So even if the hash for ESLint is OK it doesn't necessarily mean that all plugins are. And without help from ESLint it is very hard to know upfront which plugins ESLint will load and which configurations files (interesting are the once containing JS code).

I talked a lot to the VS Code team about this and VS Code will introduce a concept of trusted workspaces with a proper UI which will make the dialog superfluous.

@donggua-nor
Copy link

donggua-nor commented Aug 25, 2020

Yes, run the command ESLint: Reset Library Decision
i use this command but it tips command not found: ESLint:
what should i do? (installed eslint but it doesn't work )

sry my spoken English is not well, maybe there is something wrong syntax

@dbaeumer
Copy link
Member

@donggua-c that is strange. Which version have you installed?

@stevage
Copy link

stevage commented Aug 25, 2020

I'm finding this message super annoying. I work on tons of different projects, so even "once per workspace" is a lot, and really breaks my flow. It adds zero value to me. Would love to be able to suppress this globally.

@donggua-nor
Copy link

donggua-nor commented Aug 26, 2020

@donggua-c that is strange. Which version have you installed?

sry i tried it with a wrong way and i retry it successed. sry to waste ur time

@ramigs
Copy link

ramigs commented Aug 26, 2020

as suggested by @ryanblock in #1012 reverting to 2.1.6 solved it for me

@SergioKastro
Copy link

The same here , thanks for the suggestion to downgrade to 2.1.6. Now the pop message is gone.

@vscodebot vscodebot bot locked and limited conversation to collaborators Sep 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants