From 3dcf77771dbca5c4a140b8a99b76944132fa372f Mon Sep 17 00:00:00 2001
From: Einenlum <contact@einenlum.com>
Date: Sun, 21 Sep 2025 08:56:22 +0200
Subject: [PATCH 1/2] fix empty last error in FileUtils

---
 src/Utils/FileUtils.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/Utils/FileUtils.php b/src/Utils/FileUtils.php
index 3e4c38c1..fec0dfd9 100644
--- a/src/Utils/FileUtils.php
+++ b/src/Utils/FileUtils.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * This file is part of the Cloudinary PHP package.
  *
@@ -137,7 +138,7 @@ public static function safeFileOpen(string $filename, string $mode, bool $useInc
 
         if (! $fp) {
             $err = error_get_last();
-            throw new GeneralError($err['message']);
+            throw new GeneralError($err['message'] ?? 'Failed to open file: ' . $filename);
         }
 
         return $fp;

From 73e21291b8f12b69b0055df25d6ac6795f2c9a19 Mon Sep 17 00:00:00 2001
From: Einenlum <contact@einenlum.com>
Date: Sun, 21 Sep 2025 09:10:36 +0200
Subject: [PATCH 2/2] add tests

---
 tests/Unit/Utils/FileUtilsTest.php | 37 ++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/tests/Unit/Utils/FileUtilsTest.php b/tests/Unit/Utils/FileUtilsTest.php
index 32b94b84..5347ed71 100644
--- a/tests/Unit/Utils/FileUtilsTest.php
+++ b/tests/Unit/Utils/FileUtilsTest.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * This file is part of the Cloudinary PHP package.
  *
@@ -10,6 +11,7 @@
 
 namespace Cloudinary\Test\Unit\Utils;
 
+use Cloudinary\Api\Exception\GeneralError;
 use Cloudinary\FileUtils;
 use Cloudinary\Tag\ImageTag;
 use PHPUnit\Framework\TestCase;
@@ -64,4 +66,39 @@ public function testSplitFilenameExtension()
             self::assertEquals($file[1], FileUtils::splitPathFilenameExtension($file[0]));
         }
     }
+
+    public function testSafeFileOpenWithExistingFile()
+    {
+        $existingFile = __DIR__ . '/../../assets/sample.png';
+        $resource = FileUtils::safeFileOpen($existingFile, 'rb');
+
+        self::assertIsResource($resource);
+        self::assertEquals('stream', get_resource_type($resource));
+
+        fclose($resource);
+    }
+
+    public function testSafeFileOpenWithNonExistingFile()
+    {
+        $nonExistingFile = __DIR__ . '/../../assets/non_existing_file.txt';
+
+        $this->expectException(GeneralError::class);
+        FileUtils::safeFileOpen($nonExistingFile, 'rb');
+    }
+
+    public function testSafeFileOpenWithNonExistingFolder()
+    {
+        $nonExistingFile = 'foo/bar';
+
+        $this->expectException(GeneralError::class);
+        FileUtils::safeFileOpen($nonExistingFile, 'rb');
+    }
+
+    public function testSafeFileOpenWithEmptyPath()
+    {
+        $this->expectException(GeneralError::class);
+        $this->expectExceptionMessage('Path cannot be empty');
+
+        FileUtils::safeFileOpen('', 'rb');
+    }
 }