diff --git a/src/Monolog/Formatter/FluentdFormatter.php b/src/Monolog/Formatter/FluentdFormatter.php index 6b48ec94..e7b02930 100644 --- a/src/Monolog/Formatter/FluentdFormatter.php +++ b/src/Monolog/Formatter/FluentdFormatter.php @@ -70,13 +70,7 @@ class FluentdFormatter implements FormatterInterface $message['level_name'] = $record['level_name']; } - return '[' - . '"' . $tag . '"' - . ', ' - . $record['datetime']->getTimestamp() - . ', ' - . json_encode($message) - . ']'; + return json_encode(array($tag, $record['datetime']->getTimestamp(), $message)); } public function formatBatch(array $records) diff --git a/tests/Monolog/Formatter/FluentdFormatterTest.php b/tests/Monolog/Formatter/FluentdFormatterTest.php index 8476f4c7..622b2bae 100644 --- a/tests/Monolog/Formatter/FluentdFormatterTest.php +++ b/tests/Monolog/Formatter/FluentdFormatterTest.php @@ -40,8 +40,9 @@ class FluentdFormatterTest extends TestCase $formatter = new FluentdFormatter(); $this->assertEquals( - '["test", 0, {"message":"test","extra":[],"level":300,"level_name":"WARNING"}]', - $formatter->format($record)); + '["test",0,{"message":"test","extra":[],"level":300,"level_name":"WARNING"}]', + $formatter->format($record) + ); } /** @@ -54,7 +55,8 @@ class FluentdFormatterTest extends TestCase $formatter = new FluentdFormatter(true); $this->assertEquals( - '["test.error", 0, {"message":"test","extra":[]}]', - $formatter->format($record)); + '["test.error",0,{"message":"test","extra":[]}]', + $formatter->format($record) + ); } }