App\Services\Streamers\Streamer

Complexity

Total Complexity

4

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setSong()	0	10	3
A	__construct()	0	4	1

<?php

namespace App\Services\Streamers;

use App\Models\Song;

class Streamer
{
    /**
     * @var Song|string
     */
    protected $song;

    /**
     * @var string
     */
    protected $contentType;

    public function __construct()
    {
        // Turn off error reporting to make sure our stream isn't interfered.
        @error_reporting(0);

It seems like you do not handle an error condition for error_reporting(). This can introduce security issues, and is generally not recommended.

    }

    public function setSong(Song $song): void
    {
        $this->song = $song;

        abort_unless($this->song->s3_params || file_exists($this->song->path), 404);

        // Hard code the content type instead of relying on PHP's fileinfo()
        // or even Symfony's MIMETypeGuesser, since they appear to be wrong sometimes.
        if (!$this->song->s3_params) {
            $this->contentType = 'audio/'.pathinfo($this->song->path, PATHINFO_EXTENSION);
        }
    }
}