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

Downloading modules is broken #253

Closed
Cyberklabauter opened this issue Feb 19, 2022 · 15 comments
Closed

Downloading modules is broken #253

Cyberklabauter opened this issue Feb 19, 2022 · 15 comments
Assignees
Labels

Comments

@Cyberklabauter
Copy link

Hi ewerybody,

If I am trying to download the modules, I am getting the following error message: “Error preparing folder. Not empty but no previous version found!”

Modules get downloaded, but stuck in the temp folder.

If I am copy them manually from temp to modules they work, but it seems to be the old version (CAnt see the expected changes). Same if I download it form tps://github.com/ewerybody/a2.modules and extract them to the module folder.

Can you fix it?

@ewerybody ewerybody self-assigned this Feb 20, 2022
@ewerybody ewerybody added the bug label Feb 20, 2022
@ewerybody
Copy link
Owner

Ahh crap! yeah I was into this but lost track somewhere. I'm sorry :/
back on it!

@ewerybody
Copy link
Owner

ewerybody commented Feb 21, 2022

Hey @Cyberklabauter ! there is now 0.3.2 can you give it a try?

@Cyberklabauter
Copy link
Author

Hey ewerybody,

thank you! Your new package is detected as a virus (see Virus Total). Windows Defender deletes the file immediatly and does not let me recover it from quarantine. To download it, I have to disable windows defender completly. Maybe not the best for the most users 😉. I guess it is the module downloader. But this was not a issue with the prior version.

@ewerybody
Copy link
Owner

boahlek... This heuristics crap is grinding my gears. Yeah that's simply because there have been malware packages that have also been packed with PyInstaller... So they see some patterns in our executable and boom: MUST be a virus 🙈
Still I don't have a digital signature to apply to our executables :/

We talked about this here, here ..

@Cyberklabauter
Copy link
Author

So did you activate the UPX compression again? I know that antivirus companies flag UPX compression as a virus (that is why Lexiko removed it completly from the AHK_L version) and they getting more and more strictly.

I still can download a2 0.3.0 alpha without issues.

@ewerybody
Copy link
Owner

nope. --noupx when building the Python package:

rem "%pyinstaller%" --noupx --onedir -y "%scriptpath%" --distpath="%distpath%" --workpath="%buildpath%" --specpath=%here% --icon "%iconpath%"

and also /compress 0 when building AHK executables:

cmd = [Paths.ahk2exe, '/in', script, '/out', outpath, '/compress', '0', '/ahk', Paths.ahkexe]

@ewerybody
Copy link
Owner

Oh. See this commend on stackoverflow.

@Cyberklabauter
Copy link
Author

Is cx_Freeze a solution which works?

@ewerybody
Copy link
Owner

🤷‍♀️ Never used it. It'd be a tremendous amount of work to put it in place just to see that it doesn't.

@ewerybody
Copy link
Owner

Before that I'd try to make builds work from within Github Actions.

@ewerybody
Copy link
Owner

So did you activate the UPX compression again? I know that antivirus companies flag UPX compression as a virus (that is why Lexiko removed it completly from the AHK_L version) and they getting more and more strictly.

I still can download a2 0.3.0 alpha without issues.

Do you mean 0.2.0 alpha? Because 0.3.0 and 0.3.2 yield very similar results on virustotal.

@ewerybody
Copy link
Owner

OK. I have submitted the file for malware analysis to ms here: https://www.microsoft.com/en-us/wdsi/filesubmission
crossing fingers. ...

@ewerybody
Copy link
Owner

Hey @Cyberklabauter I just had to deal with embed Python environments for work and tried to use one of those as the base for an app package. It took some fiddeling but after all this is easy!

I'll make a portable package first and you can try. We might actually be able to ship WITHOUT freezing anything! 🤩
That means: There is no self-cooked executables anymore. Only the original python.exe and of course some AHK-ones.
That'll also speed up build times!

@ewerybody
Copy link
Owner

Do you mean 0.2.0 alpha? ...

btw: I got what you mean! Because of all this crap I excluded my a2 dev dir from scans. But when I put the zip out-side I also get this "Trojan:Win32/Wacatac.B!ml" detected on it but NOT the 0.3.0-one! Which is ultra weird since these are almost identical.

Well. If you wanna know what to think of these heuristics based approaches from all these vendors listed on virustotal:
Make an .ahk file with msgbox Hello!, compile and upload it for a check. I got 4 hits for "malware" 🤦‍♀️

@Cyberklabauter
Copy link
Author

Hey ewerybody,

I'll make a portable package first and you can try. We might actually be able to ship WITHOUT freezing anything! 🤩
That means: There is no self-cooked executables anymore. Only the original python.exe and of course some AHK-ones.
That'll also speed up build times!

Good news!

Well. If you wanna know what to think of these heuristics based approaches from all these vendors listed on virustotal:
Make an .ahk file with msgbox Hello!, compile and upload it for a check. I got 4 hits for "malware" 🤦‍♀️

I know. It started with UPX compression and then they got more and more strict. Anyways, I think Windows Defender should not alert, because as a standard it is on the most systems. Usually, I am able to recover files from quarantine. But in this case, it was erased completely. So normal users probably would fail to fix it.

I am quite busy the next days. But I will check out your update as soon as possible, maybe at the weekend or at the night if I am finding time. Looking forward to it!

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

No branches or pull requests

2 participants