silviooosilva / CacheerPHP

src/Helpers/FlushHelper.php

<?php

namespace Silviooosilva\CacheerPhp\Helpers;

/**
 * Class FlushHelper
 * 
 * @author Sílvio Silva <https://github.com/silviooosilva>
 * @package Silviooosilva\CacheerPhp
 * 
 * Builds deterministic file paths for last-flush timestamps per store type.
 */
class FlushHelper
{
    /**
     * Returns a path to store a last-flush timestamp.
     * @param string $storeType e.g., 'redis' or 'db'
     * @param string $identifier e.g., namespace or table name
     * @return string
     */
    public static function pathFor(string $storeType, string $identifier): string
    {
        $root = EnvHelper::getRootPath();
        $dir = $root . DIRECTORY_SEPARATOR . 'CacheerPHP' . DIRECTORY_SEPARATOR . 'Flush';
        if (!is_dir($dir)) {
            @mkdir($dir, 0755, true);

It seems like you do not handle an error condition for mkdir(). This can introduce security issues, and is generally not recommended.

        }
        $safeId = preg_replace('/[^a-zA-Z0-9_-]+/', '_', $identifier);
        return $dir . DIRECTORY_SEPARATOR . $storeType . '_' . $safeId . '.time';
    }
}