2 issues : image file type change to ’.jpeg‘, and big image file upload error. about upload HOT 9 CLOSED

• What is the exact problem you experience with renaming .jpg to .jpeg?
• Did you update the max file size in the extension setting page too, see image:

from upload.

• auto rename is not necessary and it's just wrong
• of course, i have set to 20480

from upload.

auto rename is not necessary and it's just wrong

I disagree. But feel free to offer proper argumentation so at least I have the chance to reconsider, because "not necessary" and "just wrong" doesn't tell me a thing. The reason to do so is to force actual mime types to be used. This prevents abuse of the upload feature. Feel free to read this article and tell me what you think afterward.

Related to the second thing, what error do you get. Uploads still have to be handled by the frontend, I haven't taken into account (nor done any tests with) long running uploads yet (eg movies of several hundreds megabytes).

from upload.

Mind you, there are 6 different accepted JPEG extensions (.jpg, .jpeg, .jpe, .jif, .jfif, .jfi) and they all should work the same. That's because the MIME type is all JPEG. So, no, it's not wrong and while it might not be necessary it provides some consistency for the extension (which now can look for just one file type instead of 6).

from upload.

@jordanjay29 @luceos so great thinking. i have no word to comment.

just 3.5mb jpg and i get error.

but when i use Image Upload 0.3.1, all fine.

from upload.

by the way, jpg is more popular than jpeg, why not auto change to jpg.

from upload.

Do you see any errors in your webserver log or storage/logs/flarum.log file that might relate to the upload issue?

from upload.

@DearTanker You should check upload_max_filesize & post_max_size in php.ini Also make sure you're editing the correct php.ini. There is usually one for CLI & one for php-fpm.

There are similar limits within the webserver config.

for nginx check: client_max_body_size

from upload.

I've improved error handling of the upload methods. They now verify ini-settings and throw related exceptions. Not going to change logic for jpeg file types.

from upload.