Write file to temp dir to ensure it is writable

Fixes #230
gordalina · Jul 28, 2023 · c1ae971 · c1ae971
1 parent ba4571e
commit c1ae971
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/src/CacheTool.php b/src/CacheTool.php
@@ -235,7 +235,7 @@ protected function getWritableTempDir($tempDir = null) {
         if (is_null($tempDir)) {
             $tempDirs = ['/dev/shm', '/var/run', sys_get_temp_dir()];
             foreach ($tempDirs as $dir) {
-                if (is_dir($dir) && is_writable($dir)) {
+                if ($this->isWritable($dir)) {
                     $tempDir = $dir;
                     break;
                 }
@@ -248,4 +248,19 @@ protected function getWritableTempDir($tempDir = null) {
 
         return $tempDir;
     }
+
+    protected function isWritable($tempDir) {
+        if (!is_dir($tempDir) || !is_writable($tempDir)) {
+            return false;
+        }
+
+        $tempFile = tempnam($tempDir, 'cachetool-test');
+
+        if ($tempFile === false) {
+            return false;
+        }
+
+        @unlink($tempFile);
+        return true;
+    }
 }
diff --git a/tests/CacheToolTest.php b/tests/CacheToolTest.php
@@ -74,6 +74,11 @@ public function testWithInexistentTempDir() {
         rmdir($dir);
     }
 
+    public function testWithExistentTempDir() {
+        $cachetool = new CacheTool(null);
+        $this->assertDirectoryExists($cachetool->getTempDir());
+    }
+
     public function testWithSetAdapterAndLogger()
     {
         $logger = $this->getLogger();