Page MenuHomePhabricator

Some half-broken PNGs cannot be rendered with the Jessie IM version
Closed, ResolvedPublic


Podrobnosti o chybě jsou připojeny k hlášení, které bude veřejně dostupné. Pokud se vám to nelíbí, můžete níže hlášení upravit a odstranit všechny údaje, které nechcete sdílet.

Error details:

error: could not load image from
user agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.62 Safari/537.36
screen size: 1366x768
canvas size: 1349x502
image size: 1265x2849
thumbnail size: CSS: 223x502, screen width: 223, real width: 320

This image can render fine on Stretch

Revisions and Commits

rTHMBREXT Thumbor Plugins
Restricted Differential Revision

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

There is an internal error when trying to render the thumbnail. @Gilles this is presumably thumbor related. Not sure how to debug issues like that; in the past there used to be an X-Request-Id that could be looked up in the logs, but it's not being set now, and while there is an X-MW-Thumbnail-Renderer field, trying to filter for that in logstash yields no results.

Oh I see. Thanks so much for information regarding this annoying error.

It's true that there's no convenient way to tie an error returned by thumbor to its corresponding entry in logstash. I'll file a task about that.*/logstash-2018.03.14/logback?id=AWIje8ltGqt6ZYdwCWHh&_g=(refreshInterval:(display:Off,pause:!f,value:0),time:(from:now-15m,mode:quick,to:now))

I need to look into why that particular entry doesn't have the url field, which would have made it easy to find.

It looks like ImageMagick can't convert that particular file due to data corruption.

Locally even a fairly recent version of ImageMagick (Version: ImageMagick 7.0.5-10 Q16 x86_64 2017-06-04) fails to convert this file, with the same error message. OS X does render it, albeit the bottom of the image is black. Combined with the ImageMagick error message, this suggests that the file is somewhat cut off before its end.

Hard to say if that corruption happened during the upload, you should check if the original you have exhibits the same symptoms. But there's not much we can do with files ImageMagick refuses to process.

Actually, I've just noticed that despite the error it spews and the non-zero exit code, ImageMagick (at least locally) does render a valid output PNG. We currently throw it away due to the exit code, but we could keep it.

Gilles renamed this task from Not showing my own image of nokia 3310 to ImageMagick may return errors when it actually converted what it could.Mar 14 2018, 8:20 AM
Gilles claimed this task.
Gilles triaged this task as Low priority.
Gilles added a project: Performance-Team.
Gilles added a revision: Restricted Differential Revision.Mar 14 2018, 8:59 AM

Turns out that the production version of IM we use (6.8.9-9 Q16 x86_64 2017-11-16) cannot render that particular PNG. More recent versions of IM (7.0.5-10 Q16 x86_64 2017-06-04) can. The change made is worthwhile, but this file won't be fixed until an ImageMagick upgrade happens.

Change 419172 had a related patch set uploaded (by Gilles; owner: Gilles):
[operations/debs/python-thumbor-wikimedia@master] Upgrade to 1.16

Change 419172 merged by Filippo Giunchedi:
[operations/debs/python-thumbor-wikimedia@master] Upgrade to 1.16

This should fix some images, but not the one in the task description because that's blocked on Stretch. I'll update the task info accordingly.

Gilles renamed this task from ImageMagick may return errors when it actually converted what it could to Some half-broken PNGs cannot be rendered with the Jessie IM version.Apr 3 2018, 11:24 AM
Gilles removed Gilles as the assignee of this task.
Gilles removed a project: Patch-For-Review.
Gilles updated the task description. (Show Details)
Maowtm moved this task from Pending review to Doing on the Thumbor board.
Gilles claimed this task.

Fixed by the Stretch upgrade