Refactor DownloaderDirect and DownloaderZipStream to handle file not …

…found exceptions
kiwilan · Sep 15, 2024 · 3e9f1fd · 3e9f1fd
1 parent ae19f25
commit 3e9f1fd
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/Utils/Downloader/DownloaderDirect.php b/src/Utils/Downloader/DownloaderDirect.php
@@ -51,7 +51,11 @@ public function speed(int $speed): self
      */
     public function get(): void
     {
-        $this->size = filesize($this->path);
+        try {
+            $this->size = filesize($this->path);
+        } catch (\Throwable $th) {
+            throw new \Exception("File not found: {$this->path}");
+        }
         ini_set('max_execution_time', $this->maxExecutionTime);
 
         $this->sendHeaders();

diff --git a/src/Utils/Downloader/DownloaderZipStream.php b/src/Utils/Downloader/DownloaderZipStream.php
@@ -46,6 +46,10 @@ public function get(): void
         );
 
         foreach ($this->files as $file) {
+            if (! file_exists($file->path)) {
+                throw new \Exception("File not found: {$file->path}");
+            }
+
             $zip->addFileFromPath(
                 fileName: $file->fileName,
                 path: $file->path,