Remove args from backtrace as it causes issues with monolog

camspiers · Nov 25, 2013 · f5ce3ce · f5ce3ce
1 parent 3bd7766
commit f5ce3ce
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 10 deletions.
diff --git a/src/Camspiers/LoggerBridge/BacktraceReporter/BasicBacktraceReporter.php b/src/Camspiers/LoggerBridge/BacktraceReporter/BasicBacktraceReporter.php
@@ -42,6 +42,9 @@ public function getBacktrace(\Exception $exception = null)
 
         if ($exception instanceof \Exception) {
             $backtrace = $exception->getTrace();
+            foreach ($backtrace as $index => $backtraceCall) {
+                unset($backtrace[$index]['args']);
+            }
         } elseif (version_compare(PHP_VERSION, '5.4.0') >= 0) {
             $backtrace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, $this->backtraceLimit);
             $skipLimit = true;

diff --git a/src/Camspiers/LoggerBridge/LoggerBridge.php b/src/Camspiers/LoggerBridge/LoggerBridge.php
@@ -417,10 +417,10 @@ public function errorHandler($errno, $errstr, $errfile, $errline)
             if (in_array($errno, $errorTypes)) {
                 // Log all errors regardless of type
                 $context = array(
-                    'errfile'   => $errfile,
-                    'errline'   => $errline,
-                    'request'   => $this->format($this->request),
-                    'model'     => $this->format($this->model)
+                    'file'    => $errfile,
+                    'line'    => $errline,
+                    'request' => $this->format($this->request),
+                    'model'   => $this->format($this->model)
                 );
 
                 if ($this->reportBacktrace) {
@@ -462,10 +462,10 @@ public function errorHandler($errno, $errstr, $errfile, $errline)
     public function exceptionHandler(\Exception $exception)
     {
         $context = array(
-            'errfile'   => $exception->getFile(),
-            'errline'   => $exception->getLine(),
-            'request'   => $this->format($this->request),
-            'model'     => $this->format($this->model)
+            'file'    => $exception->getFile(),
+            'line'    => $exception->getLine(),
+            'request' => $this->format($this->request),
+            'model'   => $this->format($this->model)
         );
 
         if ($this->reportBacktrace) {
@@ -501,8 +501,8 @@ public function fatalHandler()
             }
 
             $context = array(
-                'errfile'   => $error['file'],
-                'errline'   => $error['line']
+                'file' => $error['file'],
+                'line' => $error['line']
             );
 
             if ($this->reportBacktrace) {