Add Cache File mode

Author: MGatner

app/Config/Cache.php

@@ -46,6 +46,8 @@ class Cache extends BaseConfig
 	 * system.
 	 *
 	 * @var string
+	 *
+	 * @deprecated Use the driver-specific variant under $file
 	 */
 	public $storePath = WRITEPATH . 'cache/';
 
@@ -80,6 +82,20 @@ class Cache extends BaseConfig
 	 */
 	public $prefix = '';
 
+	/**
+	 * --------------------------------------------------------------------------
+	 * File settings
+	 * --------------------------------------------------------------------------
+	 * Your file storage preferences can be specified below, if you are using
+	 * the File driver.
+	 *
+	 * @var array<string, string|int|null>
+	 */
+	public $file = [
+		'storePath' => WRITEPATH . 'cache/',
+		'mode'      => 0640,
+	];
+
 	/**
 	 * -------------------------------------------------------------------------
 	 * Memcached settings

system/Cache/Handlers/FileHandler.php

@@ -34,6 +34,16 @@ class FileHandler implements CacheInterface
 	 */
 	protected $path;
 
+	/**
+	 * Mode for the stored files.
+	 * Must be chmod-safe (octal).
+	 *
+	 * @var int
+	 *
+	 * @see https://www.php.net/manual/en/function.chmod.php
+	 */
+	protected $mode;
+
 	//--------------------------------------------------------------------
 
 	/**
@@ -44,14 +54,23 @@ class FileHandler implements CacheInterface
 	 */
 	public function __construct(Cache $config)
 	{
-		$path = ! empty($config->storePath) ? $config->storePath : WRITEPATH . 'cache';
-		if (! is_really_writable($path))
+		if (isset($config->file))
+		{
+			$this->path = $config->file['storePath'] ?? WRITEPATH . 'cache';
+			$this->mode = $config->file['mode'] ?? 0640;
+		}
+		else
+		{
+			$this->path = ! empty($config->storePath) ? $config->storePath : WRITEPATH . 'cache';
+		}
+
+		if (! is_really_writable($this->path))
 		{
-			throw CacheException::forUnableToWrite($path);
+			throw CacheException::forUnableToWrite($this->path);
 		}
 
 		$this->prefix = $config->prefix ?: '';
-		$this->path   = rtrim($path, '/') . '/';
+		$this->path   = rtrim($this->path, '/') . '/';
 	}
 
 	//--------------------------------------------------------------------
@@ -105,7 +124,7 @@ public function save(string $key, $value, int $ttl = 60)
 
 		if ($this->writeFile($this->path . $key, serialize($contents)))
 		{
-			chmod($this->path . $key, 0640);
+			chmod($this->path . $key, $this->mode);
 
 			return true;
 		}

tests/system/Cache/Handlers/FileHandlerTest.php

@@ -26,13 +26,13 @@ protected function setUp(): void
 	{
 		parent::setUp();
 
-		//Initialize path
-		$this->config             = new \Config\Cache();
-		$this->config->storePath .= self::$directory;
+		// Initialize path
+		$this->config                     = new \Config\Cache();
+		$this->config->file['storePath'] .= self::$directory;
 
-		if (! is_dir($this->config->storePath))
+		if (! is_dir($this->config->file['storePath']))
 		{
-			mkdir($this->config->storePath, 0777, true);
+			mkdir($this->config->file['storePath'], 0777, true);
 		}
 
 		$this->fileHandler = new FileHandler($this->config);
@@ -41,20 +41,20 @@ protected function setUp(): void
 
 	public function tearDown(): void
 	{
-		if (is_dir($this->config->storePath))
+		if (is_dir($this->config->file['storePath']))
 		{
-			chmod($this->config->storePath, 0777);
+			chmod($this->config->file['storePath'], 0777);
 
 			foreach (self::getKeyArray() as $key)
 			{
-				if (is_file($this->config->storePath . DIRECTORY_SEPARATOR . $key))
+				if (is_file($this->config->file['storePath'] . DIRECTORY_SEPARATOR . $key))
 				{
-					chmod($this->config->storePath . DIRECTORY_SEPARATOR . $key, 0777);
-					unlink($this->config->storePath . DIRECTORY_SEPARATOR . $key);
+					chmod($this->config->file['storePath'] . DIRECTORY_SEPARATOR . $key, 0777);
+					unlink($this->config->file['storePath'] . DIRECTORY_SEPARATOR . $key);
 				}
 			}
 
-			rmdir($this->config->storePath);
+			rmdir($this->config->file['storePath']);
 		}
 	}
 
@@ -67,15 +67,15 @@ public function testNewWithNonWritablePath()
 	{
 		$this->expectException('CodeIgniter\Cache\Exceptions\CacheException');
 
-		chmod($this->config->storePath, 0444);
+		chmod($this->config->file['storePath'], 0444);
 		new FileHandler($this->config);
 	}
 
 	public function testSetDefaultPath()
 	{
-		//Initialize path
-		$config            = new \Config\Cache();
-		$config->storePath = null;
+		// Initialize path
+		$config                    = new \Config\Cache();
+		$config->file['storePath'] = null;
 
 		$this->fileHandler = new FileHandler($config);
 		$this->fileHandler->initialize();
@@ -98,7 +98,7 @@ public function testSave()
 	{
 		$this->assertTrue($this->fileHandler->save(self::$key1, 'value'));
 
-		chmod($this->config->storePath, 0444);
+		chmod($this->config->file['storePath'], 0444);
 		$this->assertFalse($this->fileHandler->save(self::$key2, 'value'));
 	}
 
@@ -173,6 +173,36 @@ public function testIsSupported()
 		$this->assertTrue($this->fileHandler->isSupported());
 	}
 
+	/**
+	 * @dataProvider modeProvider
+	 */
+	public function testSaveMode($int, $string)
+	{
+		// Initialize mode
+		$config               = new \Config\Cache();
+		$config->file['mode'] = $int;
+
+		$this->fileHandler = new FileHandler($config);
+		$this->fileHandler->initialize();
+
+		$this->fileHandler->save(self::$key1, 'value');
+
+		$file = $config->file['storePath'] . DIRECTORY_SEPARATOR . self::$key1;
+		$mode = substr(sprintf('%o', fileperms($file)), -4);
+
+		$this->assertEquals($string, $mode);
+	}
+
+	public function modeProvider()
+	{
+		return [
+			[0640, '0640'],
+			[0600, '0600'],
+			[0660, '0660'],
+			[0777, '0777'],
+		];
+	}
+
 	//--------------------------------------------------------------------
 
 	public function testFileHandler()
@@ -200,8 +230,8 @@ final class BaseTestFileHandler extends FileHandler
 
 	public function __construct()
 	{
-		$this->config             = new \Config\Cache();
-		$this->config->storePath .= self::$directory;
+		$this->config                     = new \Config\Cache();
+		$this->config->file['storePath'] .= self::$directory;
 
 		parent::__construct($this->config);
 	}

user_guide_src/source/libraries/caching.rst

@@ -59,9 +59,9 @@ more complex, multi-server setups.
 If you have more than one application using the same cache storage, you can add a custom prefix
 here that is prepended to all key names.
 
-**$path**
+**$file**
 
-This is used by the ``file`` handler to show where it should save the cache files to.
+This is an array of settings specific to the  ``File`` handler determine how it should save the cache files.
 
 **$memcached**
 
@@ -212,7 +212,8 @@ File-based Caching
 Unlike caching from the Output Class, the driver file-based caching
 allows for pieces of view files to be cached. Use this with care, and
 make sure to benchmark your application, as a point can come where disk
-I/O will negate positive gains by caching. This requires a writable cache directory to be really writable (0777).
+I/O will negate positive gains by caching. This requires a cache
+directory to be really writable by the application.
 
 =================
 Memcached Caching