Просмотр исходного кода

Modified ChromePhpHandler and FirePhpHandler to send no headers if no User-Agent is set

Hennadiy Verkh 11 лет назад
Родитель
Сommit
ca4daeb72b
2 измененных файлов с 11 добавлено и 5 удалено
  1. 5 2
      src/Monolog/Handler/ChromePHPHandler.php
  2. 6 3
      src/Monolog/Handler/FirePHPHandler.php

+ 5 - 2
src/Monolog/Handler/ChromePHPHandler.php

@@ -155,8 +155,11 @@ class ChromePHPHandler extends AbstractProcessingHandler
      */
     protected function headersAccepted()
     {
-        return !isset($_SERVER['HTTP_USER_AGENT'])
-               || preg_match('{\bChrome/\d+[\.\d+]*\b}', $_SERVER['HTTP_USER_AGENT']);
+        if (empty($_SERVER['HTTP_USER_AGENT'])) {
+            return false;
+        }
+
+        return preg_match('{\bChrome/\d+[\.\d+]*\b}', $_SERVER['HTTP_USER_AGENT']);
     }
 
     /**

+ 6 - 3
src/Monolog/Handler/FirePHPHandler.php

@@ -155,9 +155,12 @@ class FirePHPHandler extends AbstractProcessingHandler
      */
     protected function headersAccepted()
     {
-        return !isset($_SERVER['HTTP_USER_AGENT'])
-               || preg_match('{\bFirePHP/\d+\.\d+\b}', $_SERVER['HTTP_USER_AGENT'])
-               || isset($_SERVER['HTTP_X_FIREPHP_VERSION']);
+        if (empty($_SERVER['HTTP_USER_AGENT'])) {
+            return false;
+        }
+
+        return preg_match('{\bFirePHP/\d+\.\d+\b}', $_SERVER['HTTP_USER_AGENT'])
+        || isset($_SERVER['HTTP_X_FIREPHP_VERSION']);
     }
 
     /**