[Logging] Add support for firefox in ChromePhpHandler

arjenm · arjenm · commit c8efc4dfc2a4 · 2016-04-04T20:48:53.000+02:00
diff --git a/src/Symfony/Bridge/Monolog/Handler/ChromePhpHandler.php b/src/Symfony/Bridge/Monolog/Handler/ChromePhpHandler.php
@@ -42,7 +42,7 @@ public function onKernelResponse(FilterResponseEvent $event)
             return;
         }
 
-        if (!preg_match('{\bChrome/\d+[\.\d+]*\b}', $event->getRequest()->headers->get('User-Agent'))) {
+        if (!preg_match('{\b(?:Chrome/\d+(?:\.\d+)*|Firefox/(?:4[3-9]|[5-9]\d|\d{3,})(?:\.\d)*)\b}', $event->getRequest()->headers->get('User-Agent'))) {
             $this->sendHeaders = false;
             $this->headers = array();
 

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ public function onKernelResponse(FilterResponseEvent $event)`
`42`	`42`	`return;`
`43`	`43`	`}`
`44`	`44`
`45`		`- if (!preg_match('{\bChrome/\d+[\.\d+]*\b}', $event->getRequest()->headers->get('User-Agent'))) {`
	`45`	`+ if (!preg_match('{\b(?:Chrome/\d+(?:\.\d+)\|Firefox/(?:4[3-9]\|[5-9]\d\|\d{3,})(?:\.\d))\b}', $event->getRequest()->headers->get('User-Agent'))) {`
`46`	`46`	`$this->sendHeaders = false;`
`47`	`47`	`$this->headers = array();`
`48`	`48`