Welcome to the LimeSurvey Community Forum

Ask the community, share ideas, and connect with other LimeSurvey users!

"Abort pclzip.lib.php" error during comfort update

  • hrcittl
  • hrcittl's Avatar Topic Author
  • Offline
  • New Member
  • New Member
More
5 years 6 months ago #174459 by hrcittl
Hey there everyone,

I'm getting a "Abort pclzip.lib.php : Unzipped file is superior to upload_max_filesize or to post_max_size" on the last stage of a comfortupdate (from 3.14.8+180829 to 3.14.10 (180924)).

I've increased the size limits in both php.ini files (the CLI one, and the other one).

Still getting the error though. Any other ideas?
The topic has been locked.
  • LouisGac
  • LouisGac's Avatar
  • Visitor
  • Visitor
5 years 6 months ago #174465 by LouisGac
hi,

that's the new "zip bomb" prevention:
github.com/LimeSurvey/LimeSurvey/blob/ma...mon_helper.php#L5041

you must increase "upload_max_filesize" in your php configuration.

since many people are facing problem with this function, we will add a configuration parameter to change its values from limesurvey configuration rather than php configuration.
The topic has been locked.
  • DenisChenu
  • DenisChenu's Avatar
  • Offline
  • LimeSurvey Community Team
  • LimeSurvey Community Team
More
5 years 6 months ago #174479 by DenisChenu

LouisGac wrote: …
since many people are facing problem with this function, we will add a configuration parameter to change its values from limesurvey configuration rather than php configuration.

Since it can be forced by admin …

Else : zip bomb must not be tested for comortupdate : it's an issue here ? No ?

Assistance on LimeSurvey forum and LimeSurvey core development are on my free time.
I'm not a LimeSurvey GmbH member, professional service on demand , plugin development .
I don't answer to private message.
The topic has been locked.
More
5 years 5 months ago #176703 by Trimbos
So this cannot be fixed now otherwise then set upload_max_filesize?

Really strange order to first add something new, and then go think of a solution of how to fix what the new messes up...
The topic has been locked.
  • LouisGac
  • LouisGac's Avatar
  • Visitor
  • Visitor
5 years 5 months ago #176704 by LouisGac
In latest version, you can change it in config.php:
github.com/LimeSurvey/LimeSurvey/commit/...873f3ae940c3a91a56ce

"maximum_unzipped_size"

By default it's at 500Mb
The topic has been locked.
  • hrcittl
  • hrcittl's Avatar Topic Author
  • Offline
  • New Member
  • New Member
More
5 years 5 months ago #176720 by hrcittl
Indeed. Meanwhile, I need to update to get the new size setting so that I can change it and update. But I can't update (through comfortupdate) because I don't have the size setting.

I guess I'm going to have to manually update from github to get the setting so I can comfortupdate again. First to find the time to figure out *how* to do that, and then do it.
The topic has been locked.
More
5 years 5 months ago #176724 by jelo
Why not change the content of the two files manually?
github.com/LimeSurvey/LimeSurvey/commit/...873f3ae940c3a91a56ce

You will run into more trouble switching from comfortupdate to github and back.
So just modfiy these two files.

The meaning of the word "stable" for users
www.limesurvey.org/forum/development/117...ord-stable-for-users
The topic has been locked.
  • LouisGac
  • LouisGac's Avatar
  • Visitor
  • Visitor
5 years 5 months ago #176733 by LouisGac

hrcittl wrote: Indeed. Meanwhile, I need to update to get the new size setting so that I can change it and update. But I can't update (through comfortupdate) because I don't have the size setting.

I guess I'm going to have to manually update from github to get the setting so I can comfortupdate again. First to find the time to figure out *how* to do that, and then do it.


If you subscribe to a limeSurvey CE package I can create for you a zip file to update easily
The topic has been locked.
  • DenisChenu
  • DenisChenu's Avatar
  • Offline
  • LimeSurvey Community Team
  • LimeSurvey Community Team
More
5 years 5 months ago #176739 by DenisChenu

jelo wrote: …
So just modfiy these two files.

or just this line : github.com/LimeSurvey/LimeSurvey/commit/...e04af89a4bca062L5041
Code:
return true;
since it temporary ;)

Assistance on LimeSurvey forum and LimeSurvey core development are on my free time.
I'm not a LimeSurvey GmbH member, professional service on demand , plugin development .
I don't answer to private message.
The following user(s) said Thank You: tpartner, LouisGac, paulraper
The topic has been locked.
  • hrcittl
  • hrcittl's Avatar Topic Author
  • Offline
  • New Member
  • New Member
More
5 years 5 months ago #176810 by hrcittl
Hey! That's brilliant - nice and quick - thanks Denis!

(fyi; I had to make it return false;)

Back on track again - thanks all for your help - much appreciated!
The topic has been locked.
More
5 years 4 months ago - 5 years 4 months ago #177079 by Trimbos
It is not clear to me how to address this issue with ComfortUpdate.
I cannot access any filesystem.

"upload_max_filesize" was set to 32MB, but that was not enough.
What should it be set to then?

thnx
Last edit: 5 years 4 months ago by Trimbos.
The topic has been locked.
  • holch
  • holch's Avatar
  • Offline
  • LimeSurvey Community Team
  • LimeSurvey Community Team
More
5 years 4 months ago #177105 by holch
What do you mean with "filesystem"?

And then: is your problem exactly the same, because only then this solution mentioned here will help.

I answer at the LimeSurvey forum in my spare time, I'm not a LimeSurvey GmbH employee.
No support via private message.

The topic has been locked.

Lime-years ahead

Online-surveys for every purse and purpose