Iman\Streamer\VideoStreamer::setHeader()

Complexity

Conditions	10
Paths	22

Size

Total Lines	51
Code Lines	37

Duplication

Lines	0
Ratio	0 %

Importance

Changes	6
Bugs	4	Features	0

<?php
    private function setHeader()
    {
        ob_get_clean();
        header('Content-Type: '.$this->mime);
        header('Cache-Control: max-age=2592000, public');
        header('Expires: '.gmdate('D, d M Y H:i:s', time() + 2592000).' GMT');
        header('Last-Modified: '.gmdate('D, d M Y H:i:s', @filemtime($this->path)).' GMT');

It seems like @filemtime($this->path) can also be of type false; however, parameter $timestamp of gmdate() does only seem to accept integer|null, maybe add an additional type check?

        $this->start = 0;
        $this->size = filesize($this->path);
        $this->end = $this->size - 1;
        header('Accept-Ranges: 0-'.$this->end);

        if (! isset($_SERVER['HTTP_RANGE'])) {
            header('Content-Length: '.$this->size);

            return;
        }

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

        if (strpos($range, ',') !== false) {
            header('HTTP/1.1 416 Requested Range Not Satisfiable');
            header("Content-Range: bytes $this->start-$this->end/$this->size");
            exit;
        }

        if ($range == '-') {
            $c_start = $this->size - substr($range, 1);
        } else {
            $range = explode('-', $range);
            $c_start = $range[0];

            $c_end = (isset($range[1]) && is_numeric($range[1])) ? $range[1] : $c_end;
        }
        $c_end = ($c_end > $this->end) ? $this->end : $c_end;
        if ($c_start > $c_end || $c_start > $this->size - 1 || $c_end >= $this->size) {
            header('HTTP/1.1 416 Requested Range Not Satisfiable');
            header("Content-Range: bytes $this->start-$this->end/$this->size");
            exit;
        }
        $this->start = $c_start;
        $this->end = $c_end;
        $length = $this->end - $this->start + 1;
        fseek($this->stream, $this->start);

It seems like $this->start can also be of type string; however, parameter $offset of fseek() does only seem to accept integer, maybe add an additional type check?

        header('HTTP/1.1 206 Partial Content');
        header('Content-Length: '.$length);
        header("Content-Range: bytes $this->start-$this->end/".$this->size);
    }