-
Notifications
You must be signed in to change notification settings - Fork 62
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
Windows Defender detects Trojan:Win32/Wacatac.B!ml #33
Comments
@develar https://groups.google.com/forum/#!topic/golang-nuts/Au1FbtTZzbk says its about the ldflags. |
YES! I also updated the old topic #32 by the way just tried electron-packager as an alternative do not have this kind of issue but we want to use electron-builder instead it's more advanced and promising but with this Trojan alerts it's impossible to distribute our app... @develar before the close the topic, please do not forget to test! it's taking only 10 seconds to scan files on virustotal not hard to do... Updated to attach some screenshots... This time there are 3 files in Ann all of them mentioned as TROJAN... ... |
@nicely The issue Was closed. I opened a new one to bring it back in mind. |
To confirm, this happens with [email protected] virus on Windows 10.0.18362. |
@jrcharney Is it only the |
@jkoenig134 honestly, I don't have a definitive answer. I looked in the folder, and this is all I see. |
The copy file has been deleted by Windows Defender. What happens if you right click |
The ZIP includes the copy file. |
Should it have it? |
@develar Could you please provide a new version without the |
More viruses detected. I refuse to develop an app that includes Malware in it! Ditch the Zip! 😠 I am quarantining these files! |
@jrcharney if you are using this as part of a cordova electron app you could use the electron nightly version and put the
That is our current workaround. |
I am not familiar with cordova electron. |
@jrcharney Calm down. I just suggest a workaround that doesn´t affect the integrity of your application. The "stable" version of cordova seems not be able to provide a working cordova-electron build. |
"Calm down"? For what? I believe I stated I have no knowledge about cordova, nor is it part of my electron app. What I have done is I have scanned both the app-builder.exe and the app-builder.exe.zip. app-builder.exe is clean, but app-builder.exe.zip is in quarantine, for obvious reasons. This issue only effects the Windows 32 version of app-builder which is part of the app-builder-bin package, like it or not. (I'm using Windows 10 on a 64-bit system, and the zip file does not exist in the win/x64 folder, only in the win/ia32 folder. Now, if there is a particular reason for why an ominous zip file is in a package that doesn't appear to be in a visible repository, I would certainly like to know. Was there a pull request that was somehow integrated into the package without thorough inspection? That seems like a certain possibility, and I'm certain the NPM and Github folks should know about this unwanted software injection that contains a Trojan. Is it so hard for @develar to remove that file from that specific directory, or not? |
It doesn not related to the flags, while the flags may affect some of the false positive detections, the case here is completely different, it looks like the binaries are something different from what can be built from the repo...
|
Re-built the binary on a different machine using go 1.14.2: (1 engine warning) |
@develar when will you publish 3.5.9? Edit: 3.5.9 is available and i can confirm, that the problem is solved. |
How is it solved? The new published binary still have 4 engines warnings on virustotal even if ldfalgs weren't used. If I build the binary same way I'm getting 1 detection, which is worring... |
@fedot Windows Defender is silent. Thats enough for my needs. |
👍 Few notes:
|
Each user now pays for it — size of 386 binary increased by 6MB. Compiler flag to remove unused symbols was unset for 386 due to MS Defender bug. |
@fedot Maybe you are right and I should build binary on Windows machine. Will check next time. |
The old trojan from #32 ist gone. There´s now a new virus alert.
Virus-Total Detects these:
https://www.virustotal.com/gui/file/865ac2546cd8019ed2774e3783dcc311d35ebf962f39fd085630186e7ffa008f/detection
Windows defender this one:
The text was updated successfully, but these errors were encountered: