mirror/php-monolog
1
0
Fork 0
mirror of https://github.com/Seldaek/monolog.git synced 2025-08-01 19:00:20 +02:00
Code Issues Releases Wiki Activity

Added the support of a logging context

Browse Source
This commit is contained in:
Christophe Coevoet
2011-06-01 00:10:38 +02:00
parent a8d4fc4a34
commit c1675c59d8
13 changed files with 74 additions and 60 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/Monolog/Formatter/LineFormatter.php
View File

@@ -23,7 +23,7 @@ use Monolog\Logger;
*/
class LineFormatter implements FormatterInterface
{
const SIMPLE_FORMAT = "[%datetime%] %channel%.%level_name%: %message% %extra%\n";
const SIMPLE_FORMAT = "[%datetime%] %channel%.%level_name%: %message% %context% %extra%\n";
const SIMPLE_DATE = "Y-m-d H:i:s";
protected $format;
@@ -49,8 +49,10 @@ class LineFormatter implements FormatterInterface
$output = $this->format;
foreach ($vars['extra'] as $var => $val) {
$output = str_replace('%extra.'.$var.'%', $this->convertToString($val), $output);
unset($vars['extra'][$var]);
if (false !== strpos($output, '%extra.'.$var.'%')) {
$output = str_replace('%extra.'.$var.'%', $this->convertToString($val), $output);
unset($vars['extra'][$var]);
}
}
foreach ($vars as $var => $val) {
$output = str_replace('%'.$var.'%', $this->convertToString($val), $output);
@@ -75,7 +77,7 @@ class LineFormatter implements FormatterInterface
return (string) $data;
}
return json_encode($this->normalize($data));
return stripslashes(json_encode($this->normalize($data)));
}
protected function normalize($data)
@@ -94,6 +96,6 @@ class LineFormatter implements FormatterInterface
return $normalized;
}
return sprintf("[object] (%s: %s)", get_class($data), json_decode($data));
return sprintf("[object] (%s: %s)", get_class($data), json_encode($data));
}
}

2
src/Monolog/Formatter/WildfireFormatter.php
View File

@@ -23,7 +23,7 @@ class WildfireFormatter extends LineFormatter
/**
* Similar to LineFormatter::SIMPLE_FORMAT, except without the "[%datetime%]"
*/
const SIMPLE_FORMAT = "%channel%: %message% %extra%";
const SIMPLE_FORMAT = "%channel%: %message% %context% %extra%";
/**
* Translates Monolog log levels to Wildfire levels.

2
src/Monolog/Handler/AbstractProcessingHandler.php
View File

@@ -36,7 +36,7 @@ abstract class AbstractProcessingHandler extends AbstractHandler
$record = $this->processRecord($record);
$record['message'] = $this->getFormatter()->format($record);
$record['formatted'] = $this->getFormatter()->format($record);
$this->write($record);

2
src/Monolog/Handler/FirePHPHandler.php
View File

@@ -79,7 +79,7 @@ class FirePHPHandler extends AbstractProcessingHandler
// but we're not taking advantage of that (yet), so we're using "1" for simplicity's sake.
return $this->createHeader(
array(1, 1, 1, self::$messageIndex++),
$record['message']
$record['formatted']
);
}

6
src/Monolog/Handler/MailHandler.php
View File

@@ -33,7 +33,7 @@ abstract class MailHandler extends AbstractProcessingHandler
}
if (!empty($messages)) {
$this->send($this->getFormatter()->formatBatch($messages));
$this->send((string) $this->getFormatter()->formatBatch($messages));
}
}
@@ -49,8 +49,6 @@ abstract class MailHandler extends AbstractProcessingHandler
*/
protected function write(array $record)
{
$content = $record['message'];
$this->send($content);
$this->send((string) $record['formatted']);
}
}

2
src/Monolog/Handler/StreamHandler.php
View File

@@ -67,6 +67,6 @@ class StreamHandler extends AbstractProcessingHandler
throw new \UnexpectedValueException(sprintf('The stream or file "%s" could not be opened; it may be invalid or not writable.', $this->url));
}
}
fwrite($this->stream, (string) $record['message']);
fwrite($this->stream, (string) $record['formatted']);
}
}

2
src/Monolog/Handler/SyslogHandler.php
View File

@@ -104,6 +104,6 @@ class SyslogHandler extends AbstractProcessingHandler
*/
protected function write(array $record)
{
syslog($this->logLevels[$record['level']], (string) $record['message']);
syslog($this->logLevels[$record['level']], (string) $record['formatted']);
}
}

48
src/Monolog/Logger.php
View File

@@ -137,15 +137,17 @@ class Logger
*
* @param integer $level The logging level
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function addRecord($level, $message)
public function addRecord($level, $message, array $context = array())
{
if (!$this->handlers) {
$this->pushHandler(new StreamHandler('php://stderr', self::DEBUG));
}
$record = array(
'message' => $message,
'message' => (string) $message,
'context' => $context,
'level' => $level,
'level_name' => self::getLevelName($level),
'channel' => $this->name,
@@ -180,9 +182,10 @@ class Logger
* Adds a log record at the DEBUG level.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function addDebug($message)
public function addDebug($message, array $context = array())
{
return $this->addRecord(self::DEBUG, $message);
}
@@ -191,9 +194,10 @@ class Logger
* Adds a log record at the INFO level.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function addInfo($message)
public function addInfo($message, array $context = array())
{
return $this->addRecord(self::INFO, $message);
}
@@ -202,9 +206,10 @@ class Logger
* Adds a log record at the WARNING level.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function addWarning($message)
public function addWarning($message, array $context = array())
{
return $this->addRecord(self::WARNING, $message);
}
@@ -213,9 +218,10 @@ class Logger
* Adds a log record at the ERROR level.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function addError($message)
public function addError($message, array $context = array())
{
return $this->addRecord(self::ERROR, $message);
}
@@ -224,9 +230,10 @@ class Logger
* Adds a log record at the CRITICAL level.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function addCritical($message)
public function addCritical($message, array $context = array())
{
return $this->addRecord(self::CRITICAL, $message);
}
@@ -235,9 +242,10 @@ class Logger
* Adds a log record at the ALERT level.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function addAlert($message)
public function addAlert($message, array $context = array())
{
return $this->addRecord(self::ALERT, $message);
}
@@ -261,9 +269,10 @@ class Logger
* This method allows to have an easy ZF compatibility.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function debug($message)
public function debug($message, array $context = array())
{
return $this->addRecord(self::DEBUG, $message);
}
@@ -274,9 +283,10 @@ class Logger
* This method allows to have an easy ZF compatibility.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function info($message)
public function info($message, array $context = array())
{
return $this->addRecord(self::INFO, $message);
}
@@ -287,9 +297,10 @@ class Logger
* This method allows to have an easy ZF compatibility.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function notice($message)
public function notice($message, array $context = array())
{
return $this->addRecord(self::INFO, $message);
}
@@ -300,9 +311,10 @@ class Logger
* This method allows to have an easy ZF compatibility.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function warn($message)
public function warn($message, array $context = array())
{
return $this->addRecord(self::WARNING, $message);
}
@@ -313,9 +325,10 @@ class Logger
* This method allows to have an easy ZF compatibility.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function err($message)
public function err($message, array $context = array())
{
return $this->addRecord(self::ERROR, $message);
}
@@ -326,9 +339,10 @@ class Logger
* This method allows to have an easy ZF compatibility.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function crit($message)
public function crit($message, array $context = array())
{
return $this->addRecord(self::CRITICAL, $message);
}
@@ -339,9 +353,10 @@ class Logger
* This method allows to have an easy ZF compatibility.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function alert($message)
public function alert($message, array $context = array())
{
return $this->addRecord(self::ALERT, $message);
}
@@ -352,9 +367,10 @@ class Logger
* This method allows to have an easy ZF compatibility.
*
* @param string $message The log message
* @param array $context The log context
* @return Boolean Whether the record has been processed
*/
public function emerg($message)
public function emerg($message, array $context = array())
{
return $this->addRecord(self::ALERT, $message);
}

1
tests/Monolog/Formatter/JsonFormatterTest.php
View File

@@ -21,6 +21,7 @@ class JsonFormatterTest extends \PHPUnit_Framework_TestCase
$message = $formatter->format($msg = array(
'level_name' => 'WARNING',
'channel' => 'log',
'context' => array(),
'message' => array('foo'),
'datetime' => new \DateTime,
'extra' => array(),

20
tests/Monolog/Formatter/LineFormatterTest.php
View File

@@ -21,27 +21,29 @@ class LineFormatterTest extends \PHPUnit_Framework_TestCase
$message = $formatter->format(array(
'level_name' => 'WARNING',
'channel' => 'log',
'context' => array(),
'message' => 'foo',
'datetime' => new \DateTime,
'extra' => array(),
));
$this->assertEquals('['.date('Y-m-d').'] log.WARNING: foo '."\n", $message);
$this->assertEquals('['.date('Y-m-d').'] log.WARNING: foo [] []'."\n", $message);
}
public function testDefFormatWithArray()
public function testDefFormatWithArrayContext()
{
$formatter = new LineFormatter(null, 'Y-m-d');
$message = $formatter->format(array(
'level_name' => 'ERROR',
'channel' => 'meh',
'message' => 'foo',
'datetime' => new \DateTime,
'extra' => array(),
'message' => array(
'context' => array(
'foo' => 'bar',
'baz' => 'qux',
)
));
$this->assertEquals('['.date('Y-m-d').'] meh.ERROR: message(foo: bar, baz: qux) '."\n", $message);
$this->assertEquals('['.date('Y-m-d').'] meh.ERROR: foo {"foo":"bar","baz":"qux"} []'."\n", $message);
}
public function testDefFormatExtras()
@@ -50,11 +52,12 @@ class LineFormatterTest extends \PHPUnit_Framework_TestCase
$message = $formatter->format(array(
'level_name' => 'ERROR',
'channel' => 'meh',
'context' => array(),
'datetime' => new \DateTime,
'extra' => array('ip' => '127.0.0.1'),
'message' => 'log',
));
$this->assertEquals('['.date('Y-m-d').'] meh.ERROR: log extra(ip: 127.0.0.1)'."\n", $message);
$this->assertEquals('['.date('Y-m-d').'] meh.ERROR: log [] {"ip":"127.0.0.1"}'."\n", $message);
}
public function testDefFormatWithObject()
@@ -63,11 +66,12 @@ class LineFormatterTest extends \PHPUnit_Framework_TestCase
$message = $formatter->format(array(
'level_name' => 'ERROR',
'channel' => 'meh',
'context' => array(),
'datetime' => new \DateTime,
'extra' => array('foo' => new TestFoo, 'bar' => new TestBar, 'baz' => array()),
'message' => 'foobar',
));
$this->assertEquals('['.date('Y-m-d').'] meh.ERROR: foobar extra(foo: O:25:"Monolog\\Formatter\\TestFoo":1:{s:3:"foo";s:3:"foo";}, bar: bar, baz: a:0:{})'."\n", $message);
$this->assertEquals('['.date('Y-m-d').'] meh.ERROR: foobar [] {"foo":"[object] (Monolog\\Formatter\\TestFoo: {"foo":"foo"})","bar":"[object] (Monolog\\Formatter\\TestBar: {})","baz":[]}'."\n", $message);
}
public function testBatchFormat()
@@ -78,6 +82,7 @@ class LineFormatterTest extends \PHPUnit_Framework_TestCase
'level_name' => 'CRITICAL',
'channel' => 'test',
'message' => 'bar',
'context' => array(),
'datetime' => new \DateTime,
'extra' => array(),
),
@@ -85,11 +90,12 @@ class LineFormatterTest extends \PHPUnit_Framework_TestCase
'level_name' => 'WARNING',
'channel' => 'log',
'message' => 'foo',
'context' => array(),
'datetime' => new \DateTime,
'extra' => array(),
),
));
$this->assertEquals('['.date('Y-m-d').'] test.CRITICAL: bar '."\n".'['.date('Y-m-d').'] log.WARNING: foo '."\n", $message);
$this->assertEquals('['.date('Y-m-d').'] test.CRITICAL: bar [] []'."\n".'['.date('Y-m-d').'] log.WARNING: foo [] []'."\n", $message);
}
}

24
tests/Monolog/Formatter/WildfireFormatterTest.php
View File

@@ -15,34 +15,24 @@ use Monolog\Logger;
class WildfireFormatterTest extends \PHPUnit_Framework_TestCase
{
/**
* @dataProvider recordProvider
*/
public function testDefaultFormatIsLineFormatterWithoutNewLine($record)
public function testDefaultFormatIsLineFormatterWithoutNewLine()
{
$wildfire = new WildfireFormatter();
$message = $wildfire->format($record);
$this->assertEquals(
'70|[{"Type":"ERROR","File":"","Line":""},"meh: log extra(ip: 127.0.0.1)"]|',
$message
);
}
public function recordProvider()
{
$record = array(
'level' => Logger::ERROR,
'level_name' => 'ERROR',
'channel' => 'meh',
'context' => array(),
'datetime' => new \DateTime,
'extra' => array('ip' => '127.0.0.1'),
'message' => 'log',
);
return array(
array($record),
$message = $wildfire->format($record);
$this->assertEquals(
'75|[{"Type":"ERROR","File":"","Line":""},"meh: log [] {\\"ip\\":\\"127.0.0.1\\"}"]|',
$message
);
}
}

12
tests/Monolog/Handler/FirePHPHandlerTest.php
View File

@@ -31,8 +31,8 @@ class FirePHPHandlerTest extends TestCase
'X-Wf-Protocol-1' => 'http://meta.wildfirehq.org/Protocol/JsonStream/0.2',
'X-Wf-1-Structure-1' => 'http://meta.firephp.org/Wildfire/Structure/FirePHP/FirebugConsole/0.1',
'X-Wf-1-Plugin-1' => 'http://meta.firephp.org/Wildfire/Plugin/FirePHP/Library-FirePHPCore/0.3',
'X-Wf-1-1-1-1' => '50|[{"Type":"LOG","File":"","Line":""},"test: test "]|',
'X-Wf-1-1-1-2' => '51|[{"Type":"WARN","File":"","Line":""},"test: test "]|',
'X-Wf-1-1-1-1' => '55|[{"Type":"LOG","File":"","Line":""},"test: test [] []"]|',
'X-Wf-1-1-1-2' => '56|[{"Type":"WARN","File":"","Line":""},"test: test [] []"]|',
);
$this->assertEquals($expected, $handler->getHeaders());
@@ -52,13 +52,13 @@ class FirePHPHandlerTest extends TestCase
'X-Wf-Protocol-1' => 'http://meta.wildfirehq.org/Protocol/JsonStream/0.2',
'X-Wf-1-Structure-1' => 'http://meta.firephp.org/Wildfire/Structure/FirePHP/FirebugConsole/0.1',
'X-Wf-1-Plugin-1' => 'http://meta.firephp.org/Wildfire/Plugin/FirePHP/Library-FirePHPCore/0.3',
'X-Wf-1-1-1-1' => '50|[{"Type":"LOG","File":"","Line":""},"test: test "]|',
'X-Wf-1-1-1-2' => '51|[{"Type":"WARN","File":"","Line":""},"test: test "]|',
'X-Wf-1-1-1-1' => '55|[{"Type":"LOG","File":"","Line":""},"test: test [] []"]|',
'X-Wf-1-1-1-2' => '56|[{"Type":"WARN","File":"","Line":""},"test: test [] []"]|',
);
$expected2 = array(
'X-Wf-1-1-1-3' => '50|[{"Type":"LOG","File":"","Line":""},"test: test "]|',
'X-Wf-1-1-1-4' => '51|[{"Type":"WARN","File":"","Line":""},"test: test "]|',
'X-Wf-1-1-1-3' => '55|[{"Type":"LOG","File":"","Line":""},"test: test [] []"]|',
'X-Wf-1-1-1-4' => '56|[{"Type":"WARN","File":"","Line":""},"test: test [] []"]|',
);
$this->assertEquals($expected, $handler->getHeaders());

1
tests/Monolog/TestCase.php
View File

@@ -20,6 +20,7 @@ class TestCase extends \PHPUnit_Framework_TestCase
{
return array(
'message' => $message,
'context' => array(),
'level' => $level,
'level_name' => Logger::getLevelName($level),
'channel' => 'test',