share handler instead of client to ensure responsereceived event is sent

Author: sdkawata

src/Illuminate/Http/Client/PendingRequest.php

@@ -790,20 +790,32 @@ protected function populateResponse(Response $response)
      */
     public function buildClient()
     {
-        return $this->client = $this->client ?: new Client([
-            'handler' => $this->buildHandlerStack(),
+        return $this->client = $this->client ?: $this->createClient($this->buildHandlerStack());
+    }
+
+    /**
+     * Create new Guzzle client.
+     *
+     * @param  \GuzzleHttp\HandlerStack  $handlerStack
+     * @return \GuzzleHttp\Client
+     */
+    public function createClient($handlerStack)
+    {
+        return new Client([
+            'handler' => $handlerStack,
             'cookies' => true,
         ]);
     }
 
     /**
-     * Build the before sending handler stack.
+     * add handlers to the handler stack
      *
+     * @param  \GuzzleHttp\HandlerStack  $handlerStack
      * @return \GuzzleHttp\HandlerStack
      */
-    public function buildHandlerStack()
+    public function pushHandlers($handlerStack)
     {
-        return tap(HandlerStack::create(), function ($stack) {
+        return tap($handlerStack, function ($stack) {
             $stack->push($this->buildBeforeSendingHandler());
             $stack->push($this->buildRecorderHandler());
             $stack->push($this->buildStubHandler());
@@ -814,6 +826,16 @@ public function buildHandlerStack()
         });
     }
 
+    /**
+     * Build the before sending handler stack.
+     *
+     * @return \GuzzleHttp\HandlerStack
+     */
+    public function buildHandlerStack()
+    {
+        return $this->pushHandlers(HandlerStack::create());
+    }
+
     /**
      * Build the before sending handler.
      *
@@ -1022,4 +1044,17 @@ public function setClient(Client $client)
 
         return $this;
     }
+
+    /**
+     * set the handler function
+     *
+     * @param  callable  $handler
+     * @return $this
+     */
+    public function setHandler($handler)
+    {
+        $this->client = $this->createClient($this->pushHandlers(HandlerStack::create($handler)));
+
+        return $this;
+    }
 }

src/Illuminate/Http/Client/Pool.php

@@ -2,6 +2,8 @@
 
 namespace Illuminate\Http\Client;
 
+use GuzzleHttp\Utils;
+
 /**
  * @mixin \Illuminate\Http\Client\Factory
  */
@@ -15,11 +17,11 @@ class Pool
     protected $factory;
 
     /**
-     * The client instance.
+     * The handler function for Guzzle client
      *
-     * @var \GuzzleHttp\Client
+     * @var callable
      */
-    protected $client;
+    protected $handler;
 
     /**
      * The pool of requests.
@@ -38,7 +40,7 @@ public function __construct(Factory $factory = null)
     {
         $this->factory = $factory ?: new Factory();
 
-        $this->client = $this->factory->buildClient();
+        $this->handler = Utils::chooseHandler();
     }
 
     /**
@@ -59,7 +61,7 @@ public function as(string $key)
      */
     protected function asyncRequest()
     {
-        return $this->factory->setClient($this->client)->async();
+        return $this->factory->setHandler($this->handler)->async();
     }
 
     /**