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

Don't add defender exclusion by default #1173

Closed
jyoung15 opened this issue Aug 22, 2023 · 5 comments
Closed

Don't add defender exclusion by default #1173

jyoung15 opened this issue Aug 22, 2023 · 5 comments
Labels
enhancement New feature or request

Comments

@jyoung15
Copy link

Reopening issue reported in oclif/dev-cli#433, in regards to the following section:

Section "Add %LOCALAPPDATA%\\${config.dirname} to Windows Defender exclusions (highly recommended for performance!)"
ExecShell "" '"$0"' "/C powershell -ExecutionPolicy Bypass -Command $\\"& {Add-MpPreference -ExclusionPath $\\"$LOCALAPPDATA\\${config.dirname}$\\"}$\\" -FFFeatureOff" SW_HIDE
SectionEnd

Some EDR platforms (for example Crowdstrike) will block execution when an application automatically adds an exclusion to Defender. As stated in the original issue these exclusions can open the system to new attack vectors, and this should be disabled by default.

@cristiand391
Copy link
Member

someone commented this for the Salesforce CLI (oclif CLI):
forcedotcom/cli#2298 (comment)

have you tried if that works?

@jyoung15
Copy link
Author

Thank you @cristiand391. Sorry, I did not realize this issue was already discussed for Salesforce CLI. In my case it was for boxcli. I just tested unchecking that option and it resolved the issue. Closing this issue.

@cristiand391
Copy link
Member

Reopened to track this as a feature request:
For CLI devs that generates Windows installer, would you be interested in something like oclif pack win --no-defender-exclusion? This could either completely remove the option from the installer or mark it as unchecked by default.

let us know if you have any opinion on this :)

Copy link

git2gus bot commented Nov 15, 2023

This issue has been linked to a new work item: W-14495012

@cristiand391
Copy link
Member

added support to disable this option when generating windows installers in oclif v4.0.4:
https://github.com/oclif/oclif/blob/main/CHANGELOG.md#404-2023-11-20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants