[0.17.25] Crazy bug [bad mod zip - no notice??!]

Bugs that we were not able to reproduce, and/or are waiting for more detailed info.
Post Reply
nuhll
Filter Inserter
Filter Inserter
Posts: 798
Joined: Mon Apr 04, 2016 9:48 pm
Contact:

[0.17.25] Crazy bug [bad mod zip - no notice??!]

Post by nuhll » Sat Apr 06, 2019 9:12 pm

So i came across a very dumb and annoying bug, i dont know why, but i had 3 ACTIVE (!!) mods which were failing to load (silently) over the last X hours of gameplay :cry:

I just looked in current factorio log and saw 3 lines ala:" Could not load modname.zip, Bad Zip" (or something like this)

I mean its ok. But its not OK that you dont get a notice about it. The mods showed ALWAYS perfectly fine in the mod overview (!). I deleted them and redownloaded them and everything working fine.

So please, add a notice when something like this happens and DISABLE these mods. :!:
Last edited by nuhll on Mon Apr 08, 2019 6:54 pm, edited 1 time in total.

Shogal
Burner Inserter
Burner Inserter
Posts: 18
Joined: Tue Mar 05, 2019 1:46 am
Contact:

Re: Crazy bug [bad mod zip - no notice??!]

Post by Shogal » Sun Apr 07, 2019 12:41 am

Ideally, sha1 of downloaded mods should be checked. There is sha1 field for every mod version returned by mod portal api, so if something bad happens during download (network problem or firewall software bug) causing wrong hash - this file should be discarded and download process should be re-attempted (after n failed attempts an error should be thrown) rather that silently trusting downloaded file and marking mod download as successful.

Rseding91
Factorio Staff
Factorio Staff
Posts: 9213
Joined: Wed Jun 11, 2014 5:23 am
Contact:

Re: [0.17.25] Crazy bug [bad mod zip - no notice??!]

Post by Rseding91 » Wed Apr 24, 2019 7:33 am

Thanks for the report however I can't reproduce having a mod fail to load and still showing up in the in-game mods GUI.

When a mod fails to load (the zip is corrupt in some way) the game treats it as "not a mod - some garabge zip file/unknown zip file) and ignores it.
If you want to get ahold of me I'm almost always on Discord.

nuhll
Filter Inserter
Filter Inserter
Posts: 798
Joined: Mon Apr 04, 2016 9:48 pm
Contact:

Re: [0.17.25] Crazy bug [bad mod zip - no notice??!]

Post by nuhll » Wed Apr 24, 2019 9:12 am

I should have saved these files i guess. And or log, But i ddint know it was a bug, only when i deleted them i found out it was... (because then it was working)


Is it possible to display a message to the user if "garbage" in mod directory is found? Maybe with a "ignore" option?

User avatar
ssilk
Global Moderator
Global Moderator
Posts: 10462
Joined: Tue Apr 16, 2013 10:35 pm
Contact:

Re: [0.17.25] Crazy bug [bad mod zip - no notice??!]

Post by ssilk » Fri May 17, 2019 10:21 am

I have the same bug:

Code: Select all

...
1.137 Graphics options: [Graphics quality: normal] [Video memory usage: all] [Light scale: 25%] [DXT: low-quality]
   1.137                   [Max load threads: 32] [Max texture size: 4096] [Tex.Stream.: 0] [Rotation quality: low] [Color: 16bit]
   1.432 Error ModManager.cpp:793: Mod package read error. Opening zip /Users/ssilk/Library/Application Support/factorio/mods/deadlock-beltboxes-loaders_2.1.12.zip failed: Bad zip file
   1.446 Error ModManager.cpp:793: Mod package read error. Opening zip /Users/ssilk/Library/Application Support/factorio/mods/FARL_3.1.9.zip failed: Bad zip file
   1.745 Loading mod settings ModuleInserter 4.0.5 (settings.lua)
...

Code: Select all

~/Library/Application Support/factorio/mods » zip -T FARL_3.1.9.zip                                                                                                                                                         ssilk@ssilkPro2-4
	zip warning: missing end signature--probably not a zip file (did you
	zip warning: remember to use binary mode when you transferred it?)
	zip warning: (if you are trying to read a damaged archive try -F)

zip error: Zip file structure invalid (FARL_3.1.9.zip)


~/Library/Application Support/factorio/mods » zip -T deadlock-beltboxes-loaders_2.1.12.zip                                                                                                                                  ssilk@ssilkPro2-4
	zip warning: missing end signature--probably not a zip file (did you
	zip warning: remember to use binary mode when you transferred it?)
	zip warning: (if you are trying to read a damaged archive try -F)

zip error: Zip file structure invalid (deadlock-beltboxes-loaders_2.1.12.zip)

The error seems to come from an Auto-Update. I removed the files by hand and I tried to reload them from mods.factorio.com

Code: Select all

~/Library/Application Support/factorio/mods » zip -T FARL_3.1.9.zip                                                                                                                                                         ssilk@ssilkPro2-4
test of FARL_3.1.9.zip OK
------------------------------------------------------------
~/Library/Application Support/factorio/mods » zip -T deadlock-beltboxes-loaders_2.1.12.zip  
 ssilk@ssilkPro2-4
test of deadlock-beltboxes-loaders_2.1.12.zip OK

Reason: Unknown. Some hickup on a storage?
Cool suggestion: Eatable MOUSE-pointers.
Have you used the Advanced Search today?
Need help, question? FAQ - Wiki - Forum help
I still like small signatures...

nuhll
Filter Inserter
Filter Inserter
Posts: 798
Joined: Mon Apr 04, 2016 9:48 pm
Contact:

Re: [0.17.25] Crazy bug [bad mod zip - no notice??!]

Post by nuhll » Sat May 25, 2019 11:02 pm

Did the game load the maps without notice (but without mod?)

Post Reply

Return to “Pending”

Who is online

Users browsing this forum: No registered users