How to handle errors for videos that are bigger than max file size?

[josipRajkovicFlowout]

Hi,
first of all thank you for the great package, really appreciate the work you've put in.

I have a question regarding the ->maxFileSize("1m") option.
If I use it in my implementation it does work in a sense that it doesn't download a video to my tmp folder, but it fails in the next step as there is no reported error (if ($video->getError() !== null) { doesn't equal to true as there is no error I've checked the originally generated metadata json).
On the next step I get an error 500 on $fileInfo = $video->getFile(); line in the else condition (YoutubeDl\Entity\Video::getFile(): Return value must be of type SplFileInfo, null returned is the exact error).
My implementation:

$collection = $yt->download(
            Options::create()
                ->downloadPath('/tmp/')
				->maxFileSize("1m") // Max size limit
                ->url($request->video_url)
        );

        $firstVideoProcessed = false;
        foreach ($collection->getVideos() as $video) {
            if ($firstVideoProcessed) {
                break;
            }
            if ($video->getError() !== null) { // Doesn't equal to true since there is no error being set in the parsed row from metadata file
                Log::emergency('Error with Youtube DL: User: ' . $user->id . ' Video URL: ' . $request->video_url . ' Error: ' . $video->getError());
                // Get the temporary file's path
                if ($video->get('file')) {
                    $fileInfo = $video->getFile();
                    $originalFilePath = $fileInfo->getRealPath();

                    // Delete the temporary file
                    if (file_exists($originalFilePath)) {
                        unlink($originalFilePath);
                    }
                }
                return back()->withErrors(['video' => 'An error occurred, please contact support.']);
            } else {
                $fileInfo = $video->getFile(); // Error happens here.
                // Get the original file name and path from the SplFileInfo
                $originalFileName = $fileInfo->getFilename();
                $originalFilePath = $fileInfo->getRealPath();
                $video_data = [
                    'title' => $video->getTitle(),
                    'org_file_name' => $originalFileName,
                    'org_file_path' => $originalFilePath,
                    'extension' => $fileInfo->getExtension(),
                ];
                ProcessRemoteVideo::dispatch($video_data, $user, $workspace);

                $firstVideoProcessed = true;
            }
        }

How can I properly handle an error where the video is too big so I can notify a user too?

Thank you

[norkunas]

Hey, thanks for a good explanation :)
As I've never used this option, I didn't know how this would behave.
I've updated the code and made a PR #232, will release it today :)

[josipRajkovicFlowout]

Hey, thanks for the quick support, much appreciated!

Can't wait to test it out :)

[josipRajkovicFlowout]

Works beautifully! Thanks again and keep up the good work! :)