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

chore(cli): add info about false-positive virus on Windows #1374

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

afiune
Copy link
Contributor

@afiune afiune commented Sep 8, 2023

Summary

When we upgraded to go version 1.21, Chocolatey started flagging our binaries:

https://community.chocolatey.org/packages/lacework-cli#virus

Screenshot 2023-09-08 at 11 24 25 AM

Screenshot 2023-09-08 at 11 26 59 AM

This is a common false-positive so this PR is adding this "banner" to explain:


Important Notice

Since the release of the Lacework CLI version 1.30.0 we noticed that the virus scan reports positive findings. We
have analyzed the generated binaries and we have not found any virus.

This is a common false-positive in the Golang community for Windows binaries. For details, go to https://golang.org/doc/faq#virus


Supporting Links

When we upgraded to go version `1.21`, Chocolatey started flagging our binaries:

https://community.chocolatey.org/packages/lacework-cli#virus

![Screenshot 2023-09-08 at 11 24 25 AM](https://github.com/lacework/go-sdk/assets/5712253/7a61981b-dbae-43c6-8510-52eb44a5893a)

This is a common false-positive so this PR is adding this "banner" to explain:

---

**Important Notice**

Since the release of the Lacework CLI version `1.30.0` we noticed that the virus scan reports positive findings. We
have analyzed the generated binaries and we have not found any virus.

This is a **common false-positive** in the Golang community for Windows binaries. For details, go to https://golang.org/doc/faq#virus

---

Signed-off-by: Salim Afiune Maya <afiune@lacework.net>
@afiune afiune requested a review from a team as a code owner September 8, 2023 18:26
@afiune afiune force-pushed the afiune/windows-virus-false-positive branch from d785f22 to ea2517c Compare September 8, 2023 18:26
@afiune afiune merged commit 3072d07 into main Oct 13, 2023
@afiune afiune deleted the afiune/windows-virus-false-positive branch October 13, 2023 14:59
@lacework-releng lacework-releng mentioned this pull request Oct 20, 2023
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