Merge pull request #2 from fdisotto/master

Christoph Singer · Christoph Singer · commit 4af160807cd3 · 2015-03-06T20:17:01.000+01:00
Added LOCK_EX to prevent anyone writing to the file at the same time
diff --git a/Wa72/SimpleLogger/FileLogger.php b/Wa72/SimpleLogger/FileLogger.php
@@ -2,7 +2,7 @@
 /**
  * This class is a simple file logger implementing \Psr\Log\LoggerInterface (PSR-3)
  *
- * 
+ *
  * @author Christoph Singer
  * @license MIT
  */
@@ -27,12 +27,12 @@ public function __construct($logfile)
     public function log($level, $message, array $context = array())
     {
         $logline = '[' . date('Y-m-d H:i:s') . '] ' . strtoupper($level) . ': ' . $this->interpolate($message, $context) . "\n";
-        file_put_contents($this->logfile, $logline, FILE_APPEND);
+        file_put_contents($this->logfile, $logline, FILE_APPEND | LOCK_EX);
     }
 
     /**
      * Interpolates context values into the message placeholders.
-     * 
+     *
      * This function is just copied from the example in the PSR-3 spec
      *
      */

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`	`/**`
`3`	`3`	`* This class is a simple file logger implementing \Psr\Log\LoggerInterface (PSR-3)`
`4`	`4`	`*`
`5`		`- *`
	`5`	`+ *`
`6`	`6`	`* @author Christoph Singer`
`7`	`7`	`* @license MIT`
`8`	`8`	`*/`
`@@ -27,12 +27,12 @@ public function __construct($logfile)`
`27`	`27`	`public function log($level, $message, array $context = array())`
`28`	`28`	`{`
`29`	`29`	`$logline = '[' . date('Y-m-d H:i:s') . '] ' . strtoupper($level) . ': ' . $this->interpolate($message, $context) . "\n";`
`30`		`- file_put_contents($this->logfile, $logline, FILE_APPEND);`
	`30`	`+ file_put_contents($this->logfile, $logline, FILE_APPEND \| LOCK_EX);`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`/**`
`34`	`34`	`* Interpolates context values into the message placeholders.`
`35`		`- *`
	`35`	`+ *`
`36`	`36`	`* This function is just copied from the example in the PSR-3 spec`
`37`	`37`	`*`
`38`	`38`	`*/`