File permission option for file consumer

lib/Analytics/Consumer/File.php

@@ -20,7 +20,11 @@ public function __construct($secret, $options = array()) {
 
     try {
       $this->file_handle = fopen($options["filename"], "a");
-      chmod($options["filename"], 0777);
+      if (isset($options["filepermissions"])) {
+        chmod($options["filename"], $options["filepermissions"]);
+      } else {
+        chmod($options["filename"], 0777);
+      }
     } catch (Exception $e) {
       $this->handleError($e->getCode(), $e->getMessage());
     }

test/ConsumerFileTest.php

@@ -50,11 +50,30 @@ function testProductionProblems() {
     $this->assertFalse($tracked);
   }
 
+  function testFileSecurity() {
+    $client = new Analytics_Client("testsecret",
+                          array("consumer" => "file",
+                                "filename" => $this->filename,
+                                "filepermissions" => 0700 ));
+
+    $tracked = $client->track("some_user", "File PHP Event");
+    $this->assertEquals(0700, (fileperms($this->filename) & 0777));
+  }
+
+  function testFileSecurityDefaults() {
+    $client = new Analytics_Client("testsecret",
+                          array("consumer" => "file",
+                                "filename" => $this->filename ));
+
+    $tracked = $client->track("some_user", "File PHP Event");
+    $this->assertEquals(0777, (fileperms($this->filename) & 0777));
+  }
+
   function checkWritten() {
     exec("wc -l " . $this->filename, $output);
-    $this->assertEquals($output[0], "1 " . $this->filename);
+    $this->assertEquals(trim($output[0]), "1 " . $this->filename);
     unlink($this->filename);
   }
 
 }
-?>
+?>