Improve logs for exceptions

bluelightco · Aug 6, 2024 · 2da6bdc · 2da6bdc
1 parent 477ba15
commit 2da6bdc
Showing 1 changed file with 13 additions and 17 deletions.
diff --git a/src/Http/Client/LeverClient.php b/src/Http/Client/LeverClient.php
@@ -472,22 +472,18 @@ private function checkExpandOptions(array|string $expand): bool
 
     private function handleException(Exception $e, string $method, string $endpoint, array $options = []): void
     {
-        $logDetails = [
-            'package' => 'Bluelightco\LeverPhp',
-            'method' => $method,
-            'endpoint' => $endpoint,
-            'options' => json_encode($options),
-            'response' => null,
-        ];
-
-        if ($e instanceof ClientException) {
-            $logDetails['response'] = $e->getResponse() ? $e->getResponse()->getBody()->getContents() : null;
-        }
-
-        Log::error("HTTP $method error: ".$e->getMessage(), $logDetails);
-
-        $type = $e instanceof ClientException ? 'ClientException' : 'Exception';
-
-        throw new LeverClientException("$type error executing HTTP $method. Please check the logs for more details.");
+        Log::error("HTTP $method error: ".$e->getMessage(), [
+            'message' => $e->getMessage(),
+            'package_context' => [
+                'package' => 'Bluelightco\LeverPhp',
+                'method' => $method,
+                'endpoint' => $endpoint,
+                'options' => json_encode($options),
+                'response' => ($e instanceof ClientException ? ($e->getResponse() ? $e->getResponse()->getBody()->getContents() : null) : null),
+            ],
+            'exception' => $e,
+        ]);
+
+        throw new LeverClientException("Error executing HTTP $method. Please check the logs for more details.");
     }
 }