Quality filter does not work with format filter (thumbor)

[SNagappan]

The following ignores the quality filter when converting to jpg

{domain_name}/filters:format(jpeg)/filters:quality(50)/{key}

Looking at the source code https://github.com/awslabs/serverless-image-handler/blob/master/source/image-handler/thumbor-mapping.js#L173, the quality param is added based on file extension.

My expectation is that it would be added to this.edits.jpeg when format is specified.

[beomseoklee]

@SNagappan Thanks for your feedback.

May I know what the original file format is? As far as I can test, it seems like if the original file format is PNG, than the quality does not work. I'll dig into this issue more.

[SNagappan]

@beomseoklee It didn't work with png and it also didn't work with jpeg (with no file extension). I haven't tested with other formats but in my use case I need to convert a variety of different formats to jpeg.

[fisenkodv]

@SNagappan, we have updated our solution, the issue has been fixed. If you still see the issue with the latest version (v6.0.0), please feel free to reopen the issue.

You can refer to the recent changes here.