throw on filesystem errors

jrmajor · May 4, 2024 · ab4e226 · ab4e226
1 parent fd2d9e3
commit ab4e226
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/app/Images/Image.php b/app/Images/Image.php
@@ -51,9 +51,7 @@ public static function store(LaravelFile|UploadedFile|string $file, array $attri
 
         $path = static::disk()->putFile(static::uploadPath(), $file, 'private');
 
-        if ($path === false) {
-            throw new Exception('Failed to safe a file.');
-        }
+        assert(is_string($path));
 
         /** @var static */
         return tap(

diff --git a/config/filesystems.php b/config/filesystems.php
@@ -9,15 +9,15 @@
         'local' => [
             'driver' => 'local',
             'root' => storage_path('app'),
-            'throw' => false,
+            'throw' => true,
         ],
 
         'public' => [
             'driver' => 'local',
             'root' => storage_path('app/public'),
             'url' => env('APP_URL') . '/storage',
             'visibility' => 'public',
-            'throw' => false,
+            'throw' => true,
         ],
 
         'media' => [
@@ -28,7 +28,7 @@
             'bucket' => env('S3_BUCKET_MEDIA'),
             'endpoint' => env('S3_ENDPOINT'),
             'use_path_style_endpoint' => false,
-            'throw' => false,
+            'throw' => true,
         ],
 
         'backup' => [
@@ -39,7 +39,7 @@
             'bucket' => env('S3_BUCKET_BACKUP'),
             'endpoint' => env('S3_ENDPOINT'),
             'use_path_style_endpoint' => false,
-            'throw' => false,
+            'throw' => true,
         ],
 
     ],