Spatie\PersonalDataDownload\Http\Controllers\PersonalDataDownloadController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	6	1
A	ensureAuthorizedToDownload()	0	16	4
A	responseStream()	0	26	3

<?php

namespace Spatie\PersonalDataDownload\Http\Controllers;

use Illuminate\Support\Facades\Storage;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpFoundation\StreamedResponse;

class PersonalDataDownloadController
{
    public function __invoke(string $zipFilename): StreamedResponse
    {
        $this->ensureAuthorizedToDownload($zipFilename);

        return $this->responseStream($zipFilename);
    }

    protected function ensureAuthorizedToDownload(string $zipFilename)
    {
        if (! config('personal-data-download.authentication_required')) {
            return;
        }

        if (! auth()->check()) {

The method check does only exist in Illuminate\Contracts\Auth\Guard, but not in Illuminate\Contracts\Auth\Factory.

            abort(Response::HTTP_FORBIDDEN);
        }

        [$owningUserId] = explode('-', $zipFilename);

The variable $owningUserId does not exist. Did you forget to declare it?

        if ($owningUserId != auth()->user()->id) {

The method user does only exist in Illuminate\Contracts\Auth\Guard, but not in Illuminate\Contracts\Auth\Factory.

            abort(Response::HTTP_UNAUTHORIZED);
        }
    }

    protected function responseStream(string $filename): StreamedResponse
    {
        $disk = Storage::disk(config('personal-data-download.disk'));

        if (! $disk->exists($filename)) {
            abort(404);
        }

        $downloadHeaders = [
            'Cache-Control' => 'must-revalidate, post-check=0, pre-check=0',
            'Content-Type' => 'application/zip',
            'Content-Length' => $disk->size($filename),
            'Content-Disposition' => 'attachment; filename="'.$filename.'"',
            'Pragma' => 'public',
        ];

        return response()->stream(function () use ($filename, $disk) {

The method stream does only exist in Illuminate\Contracts\Routing\ResponseFactory, but not in Illuminate\Http\Response.

            $stream = $disk->readStream($filename);

            fpassthru($stream);

            if (is_resource($stream)) {
                fclose($stream);
            }
        }, 200, $downloadHeaders);
    }
}