FriendsOfFlarum / upload

The file upload extension with insane intelligence for your Flarum forum.
https://discuss.flarum.org/d/4154
MIT License
177 stars 96 forks source link

Issues Processing Large Number of Images #232

Open johnlewissims opened 4 years ago

johnlewissims commented 4 years ago

Love this extension. You guys are crushing it.

I have a small issue. Occasionally, users need to post a large number of images at one time. The server processes the images as they're being uploaded and resizes them according to the settings in the admin panel. The problem is that the memory limit in my php.ini gets hit if you're posting more images. Meaning that the user receives an error and is forced to upload the images in smaller batches of say, 5 at a time.

It's not the end of the world, but it would be nice if this weren't the case. Obviously I could just raise the memory limit on my server, but I was hoping for a cleaner solution. Any thoughts on this?

clarkwinkelmann commented 4 years ago

Thanks for the report.

Do you know whether the issue is only happening because of the resize or does it just always happen for a large number of images even without other options enabled?

johnlewissims commented 4 years ago

Of course!

This is only happening on resize. If I turn it off, the error goes away. But then you have to deal with your AWS bucket being overloaded with massive images.

On Wed, Jul 29, 2020 at 10:09 AM Clark Winkelmann notifications@github.com wrote:

Thanks for the report.

Do you know whether the issue is only happening because of the resize or does it just always happen for a large number of images even without other options enabled?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/FriendsOfFlarum/upload/issues/232#issuecomment-665687130, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKPAU4WHOSDQXLQKRC6HN73R6AUSTANCNFSM4PLUHZJQ .