Merge pull request #61 from upmind-automation/47-remove-guzzle-histor…

…y-middleware

#47 - Remove GuzzleStack history

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 All notable changes to the package will be documented in this file.
 
+## Unreleased
+
+- Remove Guzzle stack history & request debug data (which use history)
+
 ## v4.0.1 - 2024-02-27
 
 - Fix RuleParser issue with Rule expected to be a string, while a class or callback

src/Provider/BaseCategory.php

@@ -42,13 +42,6 @@ abstract class BaseCategory implements Contract\CategoryInterface
      */
     protected $systemInfo;
 
-    /**
-     * Array containing the history of guzzle requests for this instance.
-     *
-     * @var array<Message[]>
-     */
-    protected $guzzleHistory = [];
-
     /**
      * Returns the result of a successful provision function.
      *
@@ -126,15 +119,12 @@ final public function getSystemInfo(): SystemInfo
 
     /**
      * Get a guzzle handler stack which logs requests/responses if provider is
-     * an instance of LogsDebugData and $debugLog === true. Requests and responses
-     * will also be stored in $this->guzzleHistory.
+     * an instance of LogsDebugData and $debugLog === true.
      */
     protected function getGuzzleHandlerStack(bool $debugLog = false): HandlerStack
     {
         $stack = HandlerStack::create();
 
-        $stack->push(Middleware::history($this->guzzleHistory));
-
         if (!$debugLog || !$this instanceof LogsDebugData) {
             return $stack;
         }
@@ -157,39 +147,4 @@ protected function getGuzzleHandlerStack(bool $debugLog = false): HandlerStack
 
         return $stack;
     }
-
-    /**
-     * Returns an assoc array of debug data for the last guzzle request/response
-     * for guzzle clients whose stack was obtained from `$this->getGuzzleHandlerStack()`.
-     *
-     * @return array<array<string[]>>|null
-     */
-    protected function getLastGuzzleRequestDebug(): ?array
-    {
-        /** @var Request|null $lastRequest */
-        $lastRequest = Arr::last($this->guzzleHistory)['request'] ?? null;
-        /** @var Response|null $lastResponse */
-        $lastResponse = Arr::last($this->guzzleHistory)['response'] ?? null;
-
-        if (!$lastRequest) {
-            return null;
-        }
-
-        $debug = [
-            'last_request' => [
-                'method' => $lastRequest->getMethod(),
-                'url' => $lastRequest->getUri()->__toString(),
-            ],
-            'last_response' => null
-        ];
-
-        if ($lastResponse) {
-            $debug['last_response'] = [
-                'http_code' => $lastResponse->getStatusCode(),
-                'body' => Str::limit($lastResponse->getBody()->__toString(), 300),
-            ];
-        }
-
-        return $debug;
-    }
 }