[Logger] Stack Trace Optional (#15)

Co-authored-by: onairmarc <[email protected]>
EncoreDigitalGroup · Jul 26, 2024 · 404128b · 404128b
1 parent 9058194
commit 404128b
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 2 deletions.
diff --git a/.git-blame-ignore-revs b/.git-blame-ignore-revs
@@ -6,3 +6,4 @@ d1feb1300ad4b8237b2e41a8439a5bfa3915cf8f
 eb2793b4c08a4aae2ff7cefc61c336778a33d6a8
 5809f46c02bbe25c9799a6a4cd93c6f6ee395ece
 0c213c750e950f6e9ee4e8f2632ec41ac29a70e0
+f4e014138ae1c12a4f8a7f9778535ccb7bf1c440
diff --git a/src/Common/src/Laravel/Config/phpgenesis.php b/src/Common/src/Laravel/Config/phpgenesis.php
@@ -13,4 +13,9 @@
         'region' => env('AWS_DEFAULT_REGION', 'us-east-1'),
         'version' => '2010-12-01',
     ],
+    'logger' => [
+        'exception' => [
+            'includeStackTrace' => env('PHPGENESIS_LOGGER_EXCEPTION_INCLUDE_STACK_TRACE', true),
+        ],
+    ],
 ];
diff --git a/src/Logger/src/ExceptionLogger.php b/src/Logger/src/ExceptionLogger.php
@@ -7,6 +7,8 @@
 namespace PHPGenesis\Logger;
 
 use Exception;
+use Illuminate\Support\Facades\Config;
+use PHPGenesis\Common\Composer\Providers\Laravel;
 
 /** @experimental */
 class ExceptionLogger
@@ -82,12 +84,23 @@ private static function buildMessage(string $message): string
 
     private static function buildContext(Exception $exception, array $context = []): array
     {
-        return array_merge([
+        $mergedContext = array_merge([
             'exception.message' => $exception->getMessage(),
             'exception.code' => $exception->getCode(),
             'exception.file' => $exception->getFile(),
             'exception.line' => $exception->getLine(),
-            'exception.trace' => $exception->getTraceAsString(),
         ], $context);
+
+        if (Laravel::installed() && (Config::get('phpgenesis.logger.exception.includeStackTrace'))) {
+            $mergedContext['exception.trace'] = $exception->getTrace();
+
+            return $mergedContext;
+        }
+
+        if (!Laravel::installed()) {
+            $mergedContext['exception.trace'] = $exception->getTrace();
+        }
+
+        return $mergedContext;
     }
 }