-
Notifications
You must be signed in to change notification settings - Fork 642
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
Metadata stripped from transformed images #2034
Comments
I could see this being a config setting, or an advanced transform setting @andris-sevcenko. Probably more of a 3.0 change than 2.x though. |
Since in Craft 3 this is accomplishable with a plugin making the use of the |
For what it's worth, I'd vote for preservation of image copyright info to be a core feature, even in 2.x as it will still be some time before that version is obsolete (and it will remain relevant for clients who don't want to foot the bill of upgrading to 3.x). The rest of the stripped metadata I'm fine with being opt-in if you want to use it for something on the front-end, but copyright metadata has legal ramifications for people (chiefly photographers) who are trying to protect their work. Photographers who register their work with the US Copyright Office are afforded additional legal avenues to seek damages for copyright infringement. One of the ways they can do this is by showing that copyright information was willfully removed from their images by an infringer. But if your own website strips that information, it's one less tool you have. This issue has been a concern of mine for a while as I slowly work towards launching my own photography site, and it was also voiced to me by a current designer/photographer client who noticed his info being stripped on his site (which is already live). |
@evanwarner When the next Craft 2.6 release hits (should be today), keep an eye out for |
nice job, @andris-sevcenko -- photo people will appreciate |
Cheers @andris-sevcenko! Thanks so much, my client will be happy to hear this (as am I). |
@andris-sevcenko can you confirm this fix is working? My client's Server Info page verifies ImageMagick is installed and available. I cleared all caches, deleted transformed images and updated asset indexes. But newly transformed images still have stripped down metadata. I'm specifically looking for retention of the Artist, Copyright, Copyright Notice, Creator, and Rights metadata fields. |
@evanwarner Are you using https://craftcms.com/changelog#2-6-2993 and have you enabled the config setting? Another thing, if this is an external source, you'll probably need to re-upload the file to clear the transform source cache. If none of that works, can you send a file over to support that has these fields and loses them on transforming? |
Yep, v2.6.2993, with |
v2.6.2994 seems to have fixed it. I need to do more in-depth testing but basic copyright info is kept in place during transforms. Thanks! |
Description
Most of an image's metadata (including the important copyright info) is stripped out of an image when an asset transform is applied. This used to happen during asset uploading (see this old question, answered by @andris-sevcenko), but that has been resolved. Images downloaded directly from the CP, or loaded on a page without a transform applied still have their metadata intact.
Steps to reproduce
Additional info
The text was updated successfully, but these errors were encountered: