-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Advice to disable antivirus on Windows seems risky #354
Comments
First, you are already putting significant trust into the packages you are about to install. They can execute scripts as part of the installation with the user running the installer (sometimes a root/admin user!). This install scripts are still run even when Antivirus is turned on. There's still plenty of damage that can be done that would still pass can Antivirus scan. Second, the value of Antivirus has been debatable for years. Here's a key passage from the article just linked:
(I'm not a yarn admin, but a fellow user) |
Thanks for the comments @markstos. Besides the passage you highlighted, I think the real key passage from the article is:
I think we can agree that antivirus is not a global panacea, but I believe it is a part of a sensible security strategy and yarn should not recommend removing it. |
Any movement on this? I guess it's still not safe enough to disable your antivirus. Any solutions? I guess with WSL 2, we'll be able to |
The content for the Windows Installer page says:
This seem like incredibly risky advice for files that will be downloaded from the internet and blindly executed. Particularly as the issue could be on a deeply nested dependency that is hard to know about or verify in advance.
Is there a justification for this being safe that I am missing?
If not, I'm happy to prepare a pull request to remove this.
The text was updated successfully, but these errors were encountered: