feat: Cast uri to string in GuzzleHttpClientAspect (#640)

Co-authored-by: Deeka Wong <[email protected]>
friendsofhyperf · May 23, 2024 · 7050b3c · 7050b3c
1 parent ae3c7f4
commit 7050b3c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/Aspect/GuzzleHttpClientAspect.php b/src/Aspect/GuzzleHttpClientAspect.php
@@ -55,7 +55,7 @@ public function process(ProceedingJoinPoint $proceedingJoinPoint)
             $proceedingJoinPoint->arguments['keys']['options']['no_sentry_aspect'] = true;
         }
 
-        $uri = $arguments['keys']['uri'] ?? '';
+        $uri = (string) ($arguments['keys']['uri'] ?? '');
         $data['config'] = $guzzleConfig;
         $data['request']['method'] = $arguments['keys']['method'] ?? 'GET';
         $data['request']['options'] = $arguments['keys']['options'] ?? [];

diff --git a/src/Tracing/Aspect/GuzzleHttpClientAspect.php b/src/Tracing/Aspect/GuzzleHttpClientAspect.php
@@ -71,9 +71,8 @@ public function process(ProceedingJoinPoint $proceedingJoinPoint)
             $proceedingJoinPoint->arguments['keys']['options']['no_sentry_tracing'] = true;
         }
 
-        $uri = $arguments['uri'] ?? '/';
+        $uri = (string) ($arguments['uri'] ?? '/');
         $method = $arguments['method'] ?? 'GET';
-
         $fullUri = new \GuzzleHttp\Psr7\Uri($uri);
 
         $data = [