bug #22292 Fixes #22264 - add support for Chrome headless (redthor)

fabpot · fabpot · commit cb49907570a0 · 2017-04-05T06:41:26.000-07:00
This PR was merged into the 2.7 branch. Discussion ---------- Fixes #22264 - add support for Chrome headless | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #22264 | License | MIT | Doc PR | Commits ------- a03dbd2 Fixes #22264 - add support for Chrome headless, see also Seldaek/monolog#966
diff --git a/src/Symfony/Bridge/Monolog/Handler/ChromePhpHandler.php b/src/Symfony/Bridge/Monolog/Handler/ChromePhpHandler.php
@@ -41,7 +41,7 @@ public function onKernelResponse(FilterResponseEvent $event)
             return;
         }
 
-        if (!preg_match('{\b(?:Chrome/\d+(?:\.\d+)*|Firefox/(?:4[3-9]|[5-9]\d|\d{3,})(?:\.\d)*)\b}', $event->getRequest()->headers->get('User-Agent'))) {
+        if (!preg_match('{\b(?:Chrome/\d+(?:\.\d+)*|HeadlessChrome|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
`@@ -41,7 +41,7 @@ public function onKernelResponse(FilterResponseEvent $event)`
`41`	`41`	`return;`
`42`	`42`	`}`
`43`	`43`
`44`		`- if (!preg_match('{\b(?:Chrome/\d+(?:\.\d+)\|Firefox/(?:4[3-9]\|[5-9]\d\|\d{3,})(?:\.\d))\b}', $event->getRequest()->headers->get('User-Agent'))) {`
	`44`	`+ if (!preg_match('{\b(?:Chrome/\d+(?:\.\d+)\|HeadlessChrome\|Firefox/(?:4[3-9]\|[5-9]\d\|\d{3,})(?:\.\d))\b}', $event->getRequest()->headers->get('User-Agent'))) {`
`45`	`45`	`$this->sendHeaders = false;`
`46`	`46`	`$this->headers = array();`
`47`	`47`