uses umask function instead of custom umask

Author: bshaffer

src/Google/Cache/File.php

@@ -30,19 +30,17 @@ class Google_Cache_File implements CacheInterface
 {
   const MAX_LOCK_RETRIES = 10;
   private $path;
-  private $umask;
   private $fh;
 
   /**
    * @var use Psr\Log\LoggerInterface logger
    */
   private $logger;
 
-  public function __construct($path, LoggerInterface $logger = null, $umask = 077)
+  public function __construct($path, LoggerInterface $logger = null)
   {
     $this->path = $path;
     $this->logger = $logger;
-    $this->umask = $umask;
   }
 
   public function get($key, $expiration = false)
@@ -157,7 +155,7 @@ private function getCacheDir($file, $forWrite)
     // trim the directory separator from the path to prevent double separators
     $storageDir = rtrim($this->path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR . $dirHash;
     if ($forWrite && ! is_dir($storageDir)) {
-      if (! mkdir($storageDir, 0770 & ~$this->umask, true)) {
+      if (!mkdir($storageDir, 0777, true)) {
         $this->log(
             'error',
             'File cache creation failed',
@@ -201,7 +199,7 @@ private function acquireLock($type, $storageFile)
       return false;
     }
     if ($type == LOCK_EX) {
-      chmod($storageFile, 0660 & ~$this->umask);
+      chmod($storageFile, 0666 & ~umask());
     }
     $count = 0;
     while (!flock($this->fh, $type | LOCK_NB)) {

tests/Google/CacheTest.php

@@ -40,6 +40,20 @@ public function testFileWithTrailingSlash()
     $this->getSetDelete($cache);
   }
 
+  public function testFileDirectoryPermissions()
+  {
+    $dir = sys_get_temp_dir() . '/google-api-php-client/tests/' . rand();
+    $cache = new Google_Cache_File($dir);
+    $cache->set('foo', 'bar');
+
+    $method = new ReflectionMethod($cache, 'getWriteableCacheFile');
+    $method->setAccessible(true);
+    $filename = $method->invoke($cache, 'foo');
+    $stat = stat($dir);
+
+    $this->assertEquals(0777 & ~umask(), $stat['mode'] & 0777);
+  }
+
   public function testFileWithDefaultMask()
   {
     $dir = sys_get_temp_dir() . '/google-api-php-client/tests/';
@@ -51,21 +65,21 @@ public function testFileWithDefaultMask()
     $filename = $method->invoke($cache, 'foo');
     $stat = stat($filename);
 
-    $this->assertEquals($stat['mode'] & 0777, 0600);
+    $this->assertEquals(0666 & ~umask(), $stat['mode'] & 0777);
   }
 
   public function testFileWithCustomMask()
   {
     $dir = sys_get_temp_dir() . '/google-api-php-client/tests/';
-    $cache = new Google_Cache_File($dir, null, 07);
+    $cache = new Google_Cache_File($dir, null);
     $cache->set('foo', 'bar');
 
     $method = new ReflectionMethod($cache, 'getWriteableCacheFile');
     $method->setAccessible(true);
     $filename = $method->invoke($cache, 'foo');
     $stat = stat($filename);
 
-    $this->assertEquals($stat['mode'] & 0777, 0660);
+    $this->assertEquals(0666 & ~umask(), $stat['mode'] & 0777);
   }
 
   public function testNull()