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

File upload broken by huge images #85

Open
bergwerf opened this issue Sep 18, 2023 · 3 comments
Open

File upload broken by huge images #85

bergwerf opened this issue Sep 18, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@bergwerf
Copy link

I didn't investigate this in detail, but I uploaded a 12000x12000 pixel image by accident which broke the file uploading interface (General Data and Files). The FileCollection::edit and Image::select returned a 503, and the uploaded files were no longer displayed. I suspect the script crashes when trying to determine the dimensions of that one huge image. Perhaps worth investigating. The issue resolved itself after I manually deleted that huge image from the /shared directory.

@marcantondahmen
Copy link
Owner

Hi, thanks for reporting this. As far as I understand this could have two causes. It is either the PHP memory limit or the PHP maximum file and post size.

In both cases it can be resolved by configuring PHP correctly, allowing larger files to be uploaded and processed.

The default .htaccess includes an example configuration for large files at the top that can be uncommented. This might be already enough.

@bergwerf
Copy link
Author

Thanks for the pointer! I was thinking it could be valuable to add some kind of exception handling to report back to the user that the download handler ran out of memory (I'm a fairly tech savy user, but I was puzzled at what was happening here).

@marcantondahmen
Copy link
Owner

Yeah, I agree. A proper error message would make totally sense! Let me know if increasing the PHP settings helps.

@marcantondahmen marcantondahmen added the enhancement New feature or request label Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants