I'm seeing image_fetch() call file_transfer() even when there's no file to transfer. When this happens, Drupal eventually tries to serve a 404 page with the image/png content type, and this in turn is causing varnish to put out a 503 error. If file existence is checked first, then you just get a normal 404 as expected.

CommentFileSizeAuthor
#1 image-1702690-1.patch730 bytescatch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Active » Needs review
FileSize
image-1702690-1.patch730 bytes

Here's a patch.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented

This is arguably a core bug since file_transfer() tries to send the drupal_not_found() too, so I've opened http://drupal.org/node/1702710.

joachim’s picture

joachim CreditAttribution: joachim commented
Status: Needs review » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.