bug #22374 [Cache] Remove exception false-positive from FilesystemAda…

…pterTrait (nicolas-grekas) This PR was merged into the 3.2 branch. Discussion ---------- [Cache] Remove exception false-positive from FilesystemAdapterTrait | Q | A | ------------- | --- | Branch? | 3.2 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #20517 | License | MIT | Doc PR | - As reported in the linked issue, there seem to be a race condition in FilesystemAdapterTrait (maybe related to realpath cache?). Let's remove that exception: if the mkdir really fails, the error will be logged later on when a cache entry will be written (or succeed if the race is over.) Commits ------- a756db7 [Cache] Remove exception false-positive from FilesystemAdapterTrait
symfony · Apr 11, 2017 · bca09e4 · bca09e4
2 parents 8d34959 + a756db7
commit bca09e4
Showing 1 changed file with 7 additions and 8 deletions.
diff --git a/src/Symfony/Component/Cache/Adapter/FilesystemAdapterTrait.php b/src/Symfony/Component/Cache/Adapter/FilesystemAdapterTrait.php
@@ -27,26 +27,25 @@ private function init($namespace, $directory)
     {
         if (!isset($directory[0])) {
             $directory = sys_get_temp_dir().'/symfony-cache';
+        } else {
+            $directory = realpath($directory) ?: $directory;
         }
         if (isset($namespace[0])) {
             if (preg_match('#[^-+_.A-Za-z0-9]#', $namespace, $match)) {
                 throw new InvalidArgumentException(sprintf('Namespace contains "%s" but only characters in [-+_.A-Za-z0-9] are allowed.', $match[0]));
             }
-            $directory .= '/'.$namespace;
+            $directory .= DIRECTORY_SEPARATOR.$namespace;
         }
-        if (!file_exists($dir = $directory.'/.')) {
+        if (!file_exists($directory)) {
             @mkdir($directory, 0777, true);
         }
-        if (false === $dir = realpath($dir) ?: (file_exists($dir) ? $dir : false)) {
-            throw new InvalidArgumentException(sprintf('Cache directory does not exist (%s)', $directory));
-        }
-        $dir .= DIRECTORY_SEPARATOR;
+        $directory .= DIRECTORY_SEPARATOR;
         // On Windows the whole path is limited to 258 chars
-        if ('\\' === DIRECTORY_SEPARATOR && strlen($dir) > 234) {
+        if ('\\' === DIRECTORY_SEPARATOR && strlen($directory) > 234) {
             throw new InvalidArgumentException(sprintf('Cache directory too long (%s)', $directory));
         }
 
-        $this->directory = $dir;
+        $this->directory = $directory;
     }
 
     /**