Balloon\App\Api\Helper::streamContent()

Complexity

Conditions	18
Paths	4

Size

Total Lines

84

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	342

Importance

Changes

0

<?php
    public static function streamContent(Response $response, File $file, bool $download = false): ?Response
    {
        if (true === $download) {
            $response->setHeader('Content-Disposition', 'attachment; filename*=UTF-8\'\''.rawurlencode($file->getName()));

Consider using $file->name. There is an issue with getName() and APC-enabled PHP versions.

            $response->setHeader('Cache-Control', 'no-store, no-cache, must-revalidate, post-check=0, pre-check=0');
            $response->setHeader('Content-Type', 'application/octet-stream');
            $response->setHeader('Content-Length', (string) $file->getSize());
            $response->setHeader('Content-Transfer-Encoding', 'binary');
        } else {
            $response->setHeader('Content-Disposition', 'inline; filename*=UTF-8\'\''.rawurlencode($file->getName()));

Consider using $file->name. There is an issue with getName() and APC-enabled PHP versions.

            $response->setHeader('Content-Type', $file->getContentType());
        }

        $response->setHeader('ETag', $file->getETag());

        if (isset($_SERVER['HTTP_IF_NONE_MATCH']) && ($file->getETag() === $_SERVER['HTTP_IF_NONE_MATCH'] || $_SERVER['HTTP_IF_NONE_MATCH'] === '"*"')) {
            return $response->setCode(304);
        }

        return $response
          ->setOutputFormat(null)
          ->setBody(function () use ($file) {
              $stream = $file->get();
              $name = $file->getName();

$name is not used, you could remove the assignment.

Consider using $file->name. There is an issue with getName() and APC-enabled PHP versions.

              if (null === $stream) {
                  return;
              }

              $size = $file->getSize();
              $length = $size;
              $start = 0;
              $end = $size - 1;

              if (isset($_SERVER['HTTP_RANGE'])) {
                  header('Accept-Ranges: bytes');
                  $c_start = $start;

$c_start is not used, you could remove the assignment.

                  $c_end = $end;
                  list(, $range) = explode('=', $_SERVER['HTTP_RANGE'], 2);

                  if (strpos($range, ',') !== false) {
                      header("Content-Range: bytes $start-$end/$size");

                      throw new Exception\InvalidRange('invalid offset/limit requested');
                  }

                  if ($range == '-') {
                      $c_start = $size - substr($range, 1);
                  } else {
                      $range = explode('-', $range);
                      $c_start = $range[0];
                      $c_end = (isset($range[1]) && is_numeric($range[1])) ? $range[1] : $size;
                  }

                  $c_end = ($c_end > $end) ? $end : $c_end;
                  if ($c_start > $c_end || $c_start > $size - 1 || $c_end >= $size) {
                      header("Content-Range: bytes $start-$end/$size");

                      throw new Exception\InvalidRange('invalid offset/limit requested');
                  }

                  $start = (int) $c_start;
                  $end = (int) $c_end;
                  $length = (int) $end - $start + 1;
                  fseek($stream, $start);
                  header('HTTP/1.1 206 Partial Content');
                  header("Content-Range: bytes $start-$end/$size");
              }

              header('Content-Length: '.$length);
              $buffer = 1024 * 8;

              while (!feof($stream) && ($p = ftell($stream)) <= $end) {
                  if ($p + $buffer > $end) {
                      $buffer = $end - $p + 1;
                  }

                  echo fread($stream, $buffer);
                  flush();
              }

              fclose($stream);
          });
    }
}