hello I wrote an uploader and downloader using Laravel like this: Upload:

hello I wrote an uploader and downloader using Laravel like this: Upload:

public function __invoke(UploadRequest $request, StoreFileAction $storeFileAction) { if ($request->hasFile('file')) { $file = $request->file('file');

        $name = time() . '_' . uniqid() . '.' . $file->extension();

        $path = $file->storeAs('user-data', $name, 'public');

        $stored = $storeFileAction->execute($path);
        return Response::success('File uploaded successfully', ['dl' => route('download', ['data' => $stored->slug])], code: 201);
    }
    return Response::error('Please try again');
}

Download:

public function __invoke(Data $data) { return response()->download('storage/' . $data->path); }

Validation for file upload:

class UploadRequest extends FormRequest { public function rules(): array { return [ 'file' => ['required', 'file', 'max:1000'] ]; } }

Is this code optimal? What should be changed to be more optimal and safe from malicious users? Thank you for your advice