RequestException - check if readable before access

src/Exception/RequestException.php

@@ -126,7 +126,7 @@ public static function getResponseBodySummary(ResponseInterface $response)
     {
         $body = $response->getBody();
 
-        if (!$body->isSeekable()) {
+        if (!$body->isSeekable() || !$body->isReadable()) {
             return null;
         }
 

tests/Exception/RequestExceptionTest.php

@@ -4,6 +4,7 @@
 use GuzzleHttp\Exception\RequestException;
 use GuzzleHttp\Psr7\Request;
 use GuzzleHttp\Psr7\Response;
+use GuzzleHttp\Psr7\Stream;
 use PHPUnit\Framework\TestCase;
 
 /**
@@ -174,4 +175,30 @@ public function testObfuscateUrlWithUsernameAndPassword()
         $e = RequestException::create($r, new Response(500));
         $this->assertContains('http://user:***@www.oo.com', $e->getMessage());
     }
+
+    public function testGetResponseBodySummaryOfNonReadableStream()
+    {
+        $stream = new ReadSeekOnlyStream();
+        $stream->detach();
+        $response = new Response(500, [], $stream);
+        $this->assertNull(RequestException::getResponseBodySummary($response));
+    }
+}
+
+final class ReadSeekOnlyStream extends Stream
+{
+    public function __construct()
+    {
+        parent::__construct(fopen('php://memory', 'wb'));
+    }
+
+    public function isSeekable()
+    {
+        return true;
+    }
+
+    public function isReadable()
+    {
+        return false;
+    }
 }