[8.x] Makes the retrieval of Http client transferStats safe (#37597)

* Wraps the call to retrieve transferStats in an optional and returns an array by default.

* CS fix

* Adds support for `effectiveUri`

src/Illuminate/Http/Client/Response.php

@@ -125,11 +125,11 @@ public function status()
     /**
      * Get the effective URI of the response.
      *
-     * @return \Psr\Http\Message\UriInterface
+     * @return \Psr\Http\Message\UriInterface|null
      */
     public function effectiveUri()
     {
-        return $this->transferStats->getEffectiveUri();
+        return optional($this->transferStats)->getEffectiveUri();
     }
 
     /**
@@ -224,7 +224,7 @@ public function cookies()
      */
     public function handlerStats()
     {
-        return $this->transferStats->getHandlerStats();
+        return optional($this->transferStats)->getHandlerStats() ?? [];
     }
 
     /**

tests/Http/HttpClientTest.php

@@ -939,4 +939,16 @@ public function testTheRequestSendingAndResponseReceivedEventsAreFiredWhenAReque
 
         m::close();
     }
+
+    public function testTheTransferStatsAreCalledSafelyWhenFakingTheRequest()
+    {
+        $this->factory->fake(['https://example.com' => $this->factory->response()]);
+        $stats = $this->factory->get('https://example.com')->handlerStats();
+        $effectiveUri = $this->factory->get('https://example.com')->effectiveUri();
+
+        $this->assertIsArray($stats);
+        $this->assertEmpty($stats);
+
+        $this->assertNull($effectiveUri);
+    }
 }