Added global logging when response is not ok

Author: RoccoHoward

src/Transport/ApiClient.php

@@ -59,6 +59,11 @@ class ApiClient implements ApiClientInterface
      */
     private $endpoint;
 
+    /**
+     * @var bool
+     */
+    private $logNotOk = false;
+
     /**
      * @var PayloadSerializer
      */
@@ -97,12 +102,38 @@ public function __construct(
         $this->eventDispatcher = $eventDispatcher ?: new EventDispatcher();
     }
 
+    /**
+     * Set logging of error explanations when response is not ok.
+     *
+     * @param bool $logNotOk
+     *
+     * @return ApiClient
+     */
+    public function setLogNotOk($logNotOk)
+    {
+        $this->logNotOk = $logNotOk;
+
+        return $this;
+    }
+
+    /**
+     * Get the boolean of  logging of error explanations when response is not ok.
+     *
+     * @param string $endpoint
+     *
+     * @return string
+     */
+    public function isLogNotOk()
+    {
+        return $this->logNotOk;
+    }
+
     /**
      * Get the endpoint for this payload.
      *
      * @param string $endpoint
      *
-     * @return void
+     * @return string
      */
     public function getEndpoint()
     {
@@ -114,7 +145,7 @@ public function getEndpoint()
      *
      * @param string $endpoint
      *
-     * @return void
+     * @return ApiClient
      */
     public function setEndpoint($endpoint)
     {
@@ -126,7 +157,7 @@ public function setEndpoint($endpoint)
     /**
      * Reset the endpoint for this payload.
      *
-     * @return void
+     * @return ApiClient
      */
     public function resetEndpoint()
     {
@@ -220,6 +251,10 @@ private function doSend($method, array $data, $token = null)
 
             $this->eventDispatcher->dispatch(self::EVENT_RESPONSE, new ResponseEvent($responseData));
 
+            if ($this->isLogNotOk() && !$responseData->isOk()) {
+                error_log($responseData->getErrorExplanation());
+            }
+
             return $responseData;
         } catch (\Exception $e) {
             throw new SlackException('Failed to process response from the Slack API', null, $e);