jonathanlinat / quake-leveldesign-starterkit

A starter kit to consolidate various tools and resources into one comprehensive package, facilitating the quick and easy creation of Quake and GoldSrc engine-based maps with just a few clicks.
https://github.com/jonathanlinat/quake-leveldesign-starterkit/wiki
Creative Commons Zero v1.0 Universal
161 stars 10 forks source link

Defender goes nuts on 202231225 compile #9

Closed jkr4m3r closed 8 months ago

jkr4m3r commented 8 months ago

Doesn't look good at Virustotal either. What's going on?

jonathanlinat commented 8 months ago

Hello @jkr4m3r!

Thanks for bringing this to my attention.

I've looked into the two most recent Windows builds (20231221 & 20231225), and indeed, VirusTotal has identified some potential threats.

Currently, I'm not sure what's causing this issue, especially since QLSDK's build process is exactly the same as the one used by the tools themselves.

I'm going to investigate it further and hope to update you soon with a potential solution.

Could you please share the log file from Windows Defender or a similar tool? It would help me identify which binaries or tools are affected, as I am not a Windows user. I'll probably have to set up a Virtual Machine to conduct my tests.

jkr4m3r commented 8 months ago

deflog.txt

Snippets from Defender log attached.

jonathanlinat commented 8 months ago

I've just initiated a new release (20231230).

You can download the individual binaries from this link: https://github.com/jonathanlinat/quake-leveldesign-starterkit/actions/runs/7317584502

image

I submitted each of them to VirusTotal, and here are their respective results:

Based on your report and VirusTotal, it appears that quake-cli-tools is the set of binaries triggering the Windows Defender alert.

Interestingly, someone also mentioned the same issue with this tool on Reddit a few days ago.

I will keep investigating this matter and contact Joshua.

jonathanlinat commented 8 months ago

@jkr4m3r

After looking into the matter, I suggest adding these binaries to the Windows Defender exclusions list. This approach is common for new executables and should eventually prevent them from being flagged by the system and being deleted.

One of the possible improvements I can make in the near future is to digitally sign the recently built binaries before releasing them: https://cheapsslsecurity.com/blog/code-signing-101-how-to-sign-an-exe-or-application/

References

jonathanlinat commented 8 months ago

Another possible alternative would be to use cx_Freeze instead of PyInstaller to compile quake-cli-tools.