ImageStyleDownloadController.php 7.28 KB
Newer Older
1 2 3 4 5
<?php

namespace Drupal\image\Controller;

use Drupal\Component\Utility\Crypt;
6
use Drupal\Core\Image\ImageFactory;
7 8 9 10 11 12 13
use Drupal\Core\Lock\LockBackendInterface;
use Drupal\image\ImageStyleInterface;
use Drupal\system\FileDownloadController;
use Symfony\Component\DependencyInjection\ContainerInterface;
use Symfony\Component\HttpFoundation\BinaryFileResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
14
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
15 16 17 18 19 20
use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException;
use Symfony\Component\HttpKernel\Exception\ServiceUnavailableHttpException;

/**
 * Defines a controller to serve image styles.
 */
21
class ImageStyleDownloadController extends FileDownloadController {
22 23 24 25 26 27 28 29

  /**
   * The lock backend.
   *
   * @var \Drupal\Core\Lock\LockBackendInterface
   */
  protected $lock;

30 31 32 33 34 35 36
  /**
   * The image factory.
   *
   * @var \Drupal\Core\Image\ImageFactory
   */
  protected $imageFactory;

37 38 39 40 41 42 43
  /**
   * A logger instance.
   *
   * @var \Psr\Log\LoggerInterface
   */
  protected $logger;

44 45 46 47 48
  /**
   * Constructs a ImageStyleDownloadController object.
   *
   * @param \Drupal\Core\Lock\LockBackendInterface $lock
   *   The lock backend.
49 50
   * @param \Drupal\Core\Image\ImageFactory $image_factory
   *   The image factory.
51
   */
52
  public function __construct(LockBackendInterface $lock, ImageFactory $image_factory) {
53
    $this->lock = $lock;
54
    $this->imageFactory = $image_factory;
55
    $this->logger = $this->getLogger('image');
56 57 58 59 60 61 62 63
  }

  /**
   * {@inheritdoc}
   */
  public static function create(ContainerInterface $container) {
    return new static(
      $container->get('lock'),
64
      $container->get('image.factory')
65 66 67 68 69 70 71 72 73 74 75 76 77 78 79
    );
  }

  /**
   * Generates a derivative, given a style and image path.
   *
   * After generating an image, transfer it to the requesting agent.
   *
   * @param \Symfony\Component\HttpFoundation\Request $request
   *   The request object.
   * @param string $scheme
   *   The file scheme, defaults to 'private'.
   * @param \Drupal\image\ImageStyleInterface $image_style
   *   The image style to deliver.
   *
80 81 82
   * @return \Symfony\Component\HttpFoundation\BinaryFileResponse|\Symfony\Component\HttpFoundation\Response
   *   The transferred file as response or some error response.
   *
83 84
   * @throws \Symfony\Component\HttpKernel\Exception\NotFoundHttpException
   *   Thrown when the file request is invalid.
85 86
   * @throws \Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException
   *   Thrown when the user does not have access to the file.
87 88
   * @throws \Symfony\Component\HttpKernel\Exception\ServiceUnavailableHttpException
   *   Thrown when the file is still being generated.
89 90 91 92 93 94 95 96 97 98
   */
  public function deliver(Request $request, $scheme, ImageStyleInterface $image_style) {
    $target = $request->query->get('file');
    $image_uri = $scheme . '://' . $target;

    // Check that the style is defined, the scheme is valid, and the image
    // derivative token is valid. Sites which require image derivatives to be
    // generated without a token can set the
    // 'image.settings:allow_insecure_derivatives' configuration to TRUE to
    // bypass the latter check, but this will increase the site's vulnerability
99 100 101 102 103 104
    // to denial-of-service attacks. To prevent this variable from leaving the
    // site vulnerable to the most serious attacks, a token is always required
    // when a derivative of a style is requested.
    // The $target variable for a derivative of a style has
    // styles/<style_name>/... as structure, so we check if the $target variable
    // starts with styles/.
105
    $valid = !empty($image_style) && file_stream_wrapper_valid_scheme($scheme);
106
    if (!$this->config('image.settings')->get('allow_insecure_derivatives') || strpos(ltrim($target, '\/'), 'styles/') === 0) {
107 108 109
      $valid &= $request->query->get(IMAGE_DERIVATIVE_TOKEN) === $image_style->getPathToken($image_uri);
    }
    if (!$valid) {
110 111 112 113 114
      // Return a 404 (Page Not Found) rather than a 403 (Access Denied) as the
      // image token is for DDoS protection rather than access checking. 404s
      // are more likely to be cached (e.g. at a proxy) which enhances
      // protection from DDoS.
      throw new NotFoundHttpException();
115 116 117
    }

    $derivative_uri = $image_style->buildUri($image_uri);
118
    $headers = [];
119 120 121 122

    // If using the private scheme, let other modules provide headers and
    // control access to the file.
    if ($scheme == 'private') {
123
      $headers = $this->moduleHandler()->invokeAll('file_download', [$image_uri]);
124 125
      if (in_array(-1, $headers) || empty($headers)) {
        throw new AccessDeniedHttpException();
126 127 128 129 130
      }
    }

    // Don't try to generate file if source is missing.
    if (!file_exists($image_uri)) {
131 132 133 134 135 136 137
      // If the image style converted the extension, it has been added to the
      // original file, resulting in filenames like image.png.jpeg. So to find
      // the actual source image, we remove the extension and check if that
      // image exists.
      $path_info = pathinfo($image_uri);
      $converted_image_uri = $path_info['dirname'] . DIRECTORY_SEPARATOR . $path_info['filename'];
      if (!file_exists($converted_image_uri)) {
138
        $this->logger->notice('Source image at %source_image_path not found while trying to generate derivative image at %derivative_path.', ['%source_image_path' => $image_uri, '%derivative_path' => $derivative_uri]);
139 140 141 142 143 144
        return new Response($this->t('Error generating image, missing source file.'), 404);
      }
      else {
        // The converted file does exist, use it as the source.
        $image_uri = $converted_image_uri;
      }
145 146 147 148 149
    }

    // Don't start generating the image if the derivative already exists or if
    // generation is in progress in another thread.
    if (!file_exists($derivative_uri)) {
150
      $lock_name = 'image_style_deliver:' . $image_style->id() . ':' . Crypt::hashBase64($image_uri);
151 152 153 154
      $lock_acquired = $this->lock->acquire($lock_name);
      if (!$lock_acquired) {
        // Tell client to retry again in 3 seconds. Currently no browsers are
        // known to support Retry-After.
155
        throw new ServiceUnavailableHttpException(3, $this->t('Image generation in progress. Try again shortly.'));
156 157 158 159 160 161 162 163 164 165 166 167
      }
    }

    // Try to generate the image, unless another thread just did it while we
    // were acquiring the lock.
    $success = file_exists($derivative_uri) || $image_style->createDerivative($image_uri, $derivative_uri);

    if (!empty($lock_acquired)) {
      $this->lock->release($lock_name);
    }

    if ($success) {
168 169
      $image = $this->imageFactory->get($derivative_uri);
      $uri = $image->getSource();
170
      $headers += [
171 172
        'Content-Type' => $image->getMimeType(),
        'Content-Length' => $image->getFileSize(),
173
      ];
174 175 176 177 178
      // \Drupal\Core\EventSubscriber\FinishResponseSubscriber::onRespond()
      // sets response as not cacheable if the Cache-Control header is not
      // already modified. We pass in FALSE for non-private schemes for the
      // $public parameter to make sure we don't change the headers.
      return new BinaryFileResponse($uri, 200, $headers, $scheme !== 'private');
179 180
    }
    else {
181
      $this->logger->notice('Unable to generate the derived image located at %path.', ['%path' => $derivative_uri]);
182
      return new Response($this->t('Error generating image.'), 500);
183 184 185 186
    }
  }

}