mirror of
				https://github.com/Seldaek/monolog.git
				synced 2025-10-24 18:16:10 +02:00 
			
		
		
		
	When one use Monolog in a long process like an AMQP worker with a `FingersCrossedHandler` or `BufferHandler` there is a drawback: as soon as there is an AMQP message that generate a log >= error (for example), all next AMQP messages will output logs, even if theses messages don't generate log where level >= error. In the same context there is a drawback for processor that add an UUID to the logs. The UUID should change for each AMQP messages. --- This patch address this issue with a new interface: `ResettableInterface` interface. Side note: `reset()`, `flush()`, `clear()`, are already used in Monolog. So basically, one can use the `reset()` on the `Logger` and on some `Handler`s / `Processor`s. It's especially useful for * the `FingersCrossedHandler`: it `close()` the buffer, then it `clear()` the buffer. * the `BufferHandler`: it `flush()` the buffer, then it `clear()` the buffer. * the `UidProcessor`: it renew the `uid`.
		
			
				
	
	
		
			157 lines
		
	
	
		
			4.8 KiB
		
	
	
	
		
			PHP
		
	
	
	
	
	
			
		
		
	
	
			157 lines
		
	
	
		
			4.8 KiB
		
	
	
	
		
			PHP
		
	
	
	
	
	
| <?php
 | |
| 
 | |
| /*
 | |
|  * This file is part of the Monolog package.
 | |
|  *
 | |
|  * (c) Jordi Boggiano <j.boggiano@seld.be>
 | |
|  *
 | |
|  * For the full copyright and license information, please view the LICENSE
 | |
|  * file that was distributed with this source code.
 | |
|  */
 | |
| 
 | |
| namespace Monolog\Handler;
 | |
| 
 | |
| use Monolog\TestCase;
 | |
| use Monolog\Logger;
 | |
| 
 | |
| /**
 | |
|  * @covers Monolog\Handler\ChromePHPHandler
 | |
|  */
 | |
| class ChromePHPHandlerTest extends TestCase
 | |
| {
 | |
|     protected function setUp()
 | |
|     {
 | |
|         TestChromePHPHandler::resetStatic();
 | |
|         $_SERVER['HTTP_USER_AGENT'] = 'Monolog Test; Chrome/1.0';
 | |
|     }
 | |
| 
 | |
|     /**
 | |
|      * @dataProvider agentsProvider
 | |
|      */
 | |
|     public function testHeaders($agent)
 | |
|     {
 | |
|         $_SERVER['HTTP_USER_AGENT'] = $agent;
 | |
| 
 | |
|         $handler = new TestChromePHPHandler();
 | |
|         $handler->setFormatter($this->getIdentityFormatter());
 | |
|         $handler->handle($this->getRecord(Logger::DEBUG));
 | |
|         $handler->handle($this->getRecord(Logger::WARNING));
 | |
| 
 | |
|         $expected = array(
 | |
|             'X-ChromeLogger-Data'   => base64_encode(utf8_encode(json_encode(array(
 | |
|                 'version' => ChromePHPHandler::VERSION,
 | |
|                 'columns' => array('label', 'log', 'backtrace', 'type'),
 | |
|                 'rows' => array(
 | |
|                     'test',
 | |
|                     'test',
 | |
|                 ),
 | |
|                 'request_uri' => '',
 | |
|             )))),
 | |
|         );
 | |
| 
 | |
|         $this->assertEquals($expected, $handler->getHeaders());
 | |
|     }
 | |
| 
 | |
|     public static function agentsProvider()
 | |
|     {
 | |
|         return array(
 | |
|             array('Monolog Test; Chrome/1.0'),
 | |
|             array('Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0'),
 | |
|             array('Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/56.0.2924.76 Chrome/56.0.2924.76 Safari/537.36'),
 | |
|             array('Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome Safari/537.36'),
 | |
|         );
 | |
|     }
 | |
| 
 | |
|     public function testHeadersOverflow()
 | |
|     {
 | |
|         $handler = new TestChromePHPHandler();
 | |
|         $handler->handle($this->getRecord(Logger::DEBUG));
 | |
|         $handler->handle($this->getRecord(Logger::WARNING, str_repeat('a', 150 * 1024)));
 | |
| 
 | |
|         // overflow chrome headers limit
 | |
|         $handler->handle($this->getRecord(Logger::WARNING, str_repeat('a', 100 * 1024)));
 | |
| 
 | |
|         $expected = array(
 | |
|             'X-ChromeLogger-Data'   => base64_encode(utf8_encode(json_encode(array(
 | |
|                 'version' => ChromePHPHandler::VERSION,
 | |
|                 'columns' => array('label', 'log', 'backtrace', 'type'),
 | |
|                 'rows' => array(
 | |
|                     array(
 | |
|                         'test',
 | |
|                         'test',
 | |
|                         'unknown',
 | |
|                         'log',
 | |
|                     ),
 | |
|                     array(
 | |
|                         'test',
 | |
|                         str_repeat('a', 150 * 1024),
 | |
|                         'unknown',
 | |
|                         'warn',
 | |
|                     ),
 | |
|                     array(
 | |
|                         'monolog',
 | |
|                         'Incomplete logs, chrome header size limit reached',
 | |
|                         'unknown',
 | |
|                         'warn',
 | |
|                     ),
 | |
|                 ),
 | |
|                 'request_uri' => '',
 | |
|             )))),
 | |
|         );
 | |
| 
 | |
|         $this->assertEquals($expected, $handler->getHeaders());
 | |
|     }
 | |
| 
 | |
|     public function testConcurrentHandlers()
 | |
|     {
 | |
|         $handler = new TestChromePHPHandler();
 | |
|         $handler->setFormatter($this->getIdentityFormatter());
 | |
|         $handler->handle($this->getRecord(Logger::DEBUG));
 | |
|         $handler->handle($this->getRecord(Logger::WARNING));
 | |
| 
 | |
|         $handler2 = new TestChromePHPHandler();
 | |
|         $handler2->setFormatter($this->getIdentityFormatter());
 | |
|         $handler2->handle($this->getRecord(Logger::DEBUG));
 | |
|         $handler2->handle($this->getRecord(Logger::WARNING));
 | |
| 
 | |
|         $expected = array(
 | |
|             'X-ChromeLogger-Data'   => base64_encode(utf8_encode(json_encode(array(
 | |
|                 'version' => ChromePHPHandler::VERSION,
 | |
|                 'columns' => array('label', 'log', 'backtrace', 'type'),
 | |
|                 'rows' => array(
 | |
|                     'test',
 | |
|                     'test',
 | |
|                     'test',
 | |
|                     'test',
 | |
|                 ),
 | |
|                 'request_uri' => '',
 | |
|             )))),
 | |
|         );
 | |
| 
 | |
|         $this->assertEquals($expected, $handler2->getHeaders());
 | |
|     }
 | |
| }
 | |
| 
 | |
| class TestChromePHPHandler extends ChromePHPHandler
 | |
| {
 | |
|     protected $headers = array();
 | |
| 
 | |
|     public static function resetStatic()
 | |
|     {
 | |
|         self::$initialized = false;
 | |
|         self::$overflowed = false;
 | |
|         self::$sendHeaders = true;
 | |
|         self::$json['rows'] = array();
 | |
|     }
 | |
| 
 | |
|     protected function sendHeader($header, $content)
 | |
|     {
 | |
|         $this->headers[$header] = $content;
 | |
|     }
 | |
| 
 | |
|     public function getHeaders()
 | |
|     {
 | |
|         return $this->headers;
 | |
|     }
 | |
| }
 |