Cms: [Feature Request] Option to disable image processing completely

[benface]

Description

When uploading an image in an asset volume, Craft processes it and rewrites it, often leaving bad compression artifacts. There doesn't seem to be a way to prevent this from happening. Here's an example with defaultImageQuality set to 100 (the maximum quality):

Before upload (47 kb)

After upload (34 kb)

I suggest a new setting (this could be a general config setting or a volume setting) to disable image rewriting altogether. I understand images probably still need to be processed for security reasons, but if it doesn't contain anything suspicious, I would like Craft to store exactly the image that I uploaded.

Steps to reproduce

1. In a new Craft install, create an asset volume and set defaultImageQuality to 100.
2. Download this image: https://benface.com/files/bird.jpg
3. Upload the image in the asset volume from Craft's CP.
4. Download the uploaded image and look at it.

Additional info

• Craft version: 3.0.13.2
• PHP version: 7.1.13

[ThatGuySam]

Today Craft was doubling the size of my PNGs after I had already compressed them with tinypng.com.

It would be beautiful if I could have an option to not compress my images before I upload them. The option doesn't have to show up by default just enabled from settings(ex: Let users disable compression) and then the user gets to choose whether to use compression(checked by default)

Here are some examples of what it could look like:

From Assets

From Modal Uploader

[brandonkelly]

Craft isn’t attempting to compress them, it’s ensuring that there are no malicious scripts embedded within the image. Unfortunately the only way to do that is to reprocess it.

If you are using ImageMagick, it should do a pretty good job of keeping the file size similar to what you uploaded.

[ThatGuySam]

That makes sense. I'll look in to using ImageMagick or a plugin for now

[dgrigg]

@brandonkelly any chance of revisiting this option. I'm working on a site that is displaying high resolution, high quality images. The images are being optimized to the max to ensure minimum size with maximum quality before upload. Having the images processed during upload is causing the file sizes to nearly double, which isn't acceptable for site performance. There's an option to disable processing on SVG images, https://docs.craftcms.com/v3/config/config-settings.html#sanitizesvguploads. Could we have the same option for raster images?

[brandonkelly]

I will reopen since the main request wasn’t addressed. It opens up an attack vector though.

[ThatGuySam]

@brandonkelly You could name the option something like React's dangerouslySetInnerHTML.

Maybe something like dangerouslyDisableImageProcessing or probably something shorter than that.

[dgrigg]

@brandonkelly it would be nice to have the option with a caveat, similar to the SVG override. For trusted content entry sources it would be very nice to have.

[saatchinorway]

This is still an issue, a header PNG we uploaded grew from 106kB to 314kB and acquired strange artifacts in the logo lettering. Need to be an option to allow direct upload, we do not allow external uploads anyway.

[andris-sevcenko]

Just added the sanitizeCpImageUploads (true by default) setting for Craft 3.6, which allows to opt-out of image sanitizing for Assets uploaded via CP. This also includes the site icon and site image.

[brandonkelly]

Craft 3.6 has now been officially released ✨