Skip to content

/ harmony

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

Jump to bottom

False positive virus #19

Closed
giodelgado opened this issue Sep 8, 2016 · 3 comments
Closed

False positive virus #19

giodelgado opened this issue Sep 8, 2016 · 3 comments
Labels
bug

Comments

@giodelgado
Copy link

@giodelgado giodelgado commented Sep 8, 2016

Norton deleted the file after downloading, Virustotal.com detected this as well:

https://www.virustotal.com/en/file/3782611aad4faa7f7f4db0785302d0ebe6e628da534c4864930146eb185ef0ca/analysis/

?
@vincelwt
Copy link
Owner

@vincelwt vincelwt commented Sep 8, 2016

That's strange,
Harmony is builded for Windows with electron-builder (open-source).
Maybe an issue from their side?
@vincelwt
Copy link
Owner

@vincelwt vincelwt commented Sep 8, 2016

It seems to be caused by Asar packaging, see:
http://electron.atom.io/docs/tutorial/application-packaging/

Will try to disable it to see if it changes anything
@vincelwt vincelwt added the bug label Sep 8, 2016
@vincelwt vincelwt changed the title Virus Detected? False positive virus Sep 8, 2016
vincelwt added a commit that referenced this issue Sep 8, 2016
@vincelwt
Disable asar when build for false virus detection (#19)
1746dda
@vincelwt
Copy link
Owner

@vincelwt vincelwt commented Sep 8, 2016

Still detected:
https://www.virustotal.com/en/file/bf1d3cd11f68550d3c2bdf138ab5fc440c07850ef08987c08631ba8fa832ea53/analysis/1473345665/

Closing until news
@vincelwt vincelwt closed this Sep 8, 2016
@vincelwt vincelwt mentioned this issue Apr 1, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@vincelwt @giodelgado
You can’t perform that action at this time.