Nicely display JSON error responses

[shpub.git] / src / shpub / Request.php

diff --git a/src/shpub/Request.php b/src/shpub/Request.php
index 3a4512e3bca75f666c38569bb32a68265be890ea..32e25de04d290d42e6a94b408e728242b3afd87a 100644 (file)
--- a/src/shpub/Request.php
+++ b/src/shpub/Request.php
@@ -96,15 +96,37 @@ class Request
         $res = $this->req->send();
 
         if (intval($res->getStatus() / 100) != 2) {
-            Log::err(
-                'Server returned an error status code ' . $res->getStatus()
-            );
-            Log::err($res->getBody());
-            exit(11);
+            $this->displayErrorResponse($res);
         }
         return $res;
     }
 
+    protected function displayErrorResponse($res)
+    {
+        Log::err(
+            'Server returned an error status code ' . $res->getStatus()
+        );
+
+        $shown = false;
+        if ($res->getHeader('content-type') == 'application/json') {
+            $errData = json_decode($res->getBody());
+            if (!isset($errData->error)) {
+                Log::err('Error response does not contain "error" property');
+            } else if (isset($errData->error_description)) {
+                Log::err($errData->error . ': ' . $errData->error_description);
+                $shown = true;
+            } else {
+                Log::err($errData->error);
+                $shown = true;
+            }
+        }
+
+        if (!$shown) {
+            Log::err($res->getBody());
+        }
+        exit(11);
+    }
+
     public function setAction($action)
     {
         $this->action = $action;
@@ -154,7 +176,7 @@ class Request
     /**
      * @return string URL at media endpoint
      */
-    protected function uploadToMediaEndpoint($fileName)
+    public function uploadToMediaEndpoint($fileName)
     {
         $httpReq = $this->getHttpRequest(
             $this->host->endpoints->media, $this->host->token