From 27d2a72c18da0cbbd3a8082c86f8f2143d9cacb0 Mon Sep 17 00:00:00 2001 From: John LeSueur Date: Fri, 12 Aug 2016 15:03:09 -0600 Subject: [PATCH] Remove the chmod 777 There is no reason for this library to set permissions on the file. Sysadmins can manage what permissions should be when the file is created, and what permissions are required for the cron to read/rename/write the file. --- lib/Segment/Consumer/File.php | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/Segment/Consumer/File.php b/lib/Segment/Consumer/File.php index 8998615..9623ba6 100644 --- a/lib/Segment/Consumer/File.php +++ b/lib/Segment/Consumer/File.php @@ -20,7 +20,6 @@ public function __construct($secret, $options = array()) { try { $this->file_handle = fopen($options["filename"], "a"); - chmod($options["filename"], 0777); } catch (Exception $e) { $this->handleError($e->getCode(), $e->getMessage()); }