fix(checkins): read error message from response body (#224)

subzero10 · web-flow · commit f6fd084522a3 · 2025-01-07T17:28:00.000+02:00
diff --git a/src/Exceptions/ServiceException.php b/src/Exceptions/ServiceException.php
@@ -85,4 +85,13 @@ public static function missingPersonalAuthToken(): self
     {
         return new static("Missing personal auth token. This token is required to use Honeybadger's Data APIs.");
     }
+
+    /**
+     * @param string $message
+     * @return self
+     */
+    public static function withMessage(string $message): self
+    {
+        return new static($message);
+    }
 }
diff --git a/src/Exceptions/ServiceExceptionFactory.php b/src/Exceptions/ServiceExceptionFactory.php
@@ -27,6 +27,19 @@ public function make(bool $isFromEventsApi = false): ServiceException
 
     private function exception(bool $isFromEventsApi = false): ServiceException
     {
+        try {
+            $message = $this->response->getBody()->getContents();
+            if (!empty($message)) {
+                $data = json_decode($message, true);
+                if (isset($data['errors'])) {
+                    return ServiceException::withMessage($data['errors']);
+                }
+            }
+        } catch (\Exception $e) {
+            // Do nothing
+            // Fallback to default error messages based on status code
+        }
+
         if ($this->response->getStatusCode() === Response::HTTP_FORBIDDEN) {
             return ServiceException::invalidApiKey();
         }

Original file line number	Diff line number	Diff line change
`@@ -85,4 +85,13 @@ public static function missingPersonalAuthToken(): self`
`85`	`85`	`{`
`86`	`86`	`return new static("Missing personal auth token. This token is required to use Honeybadger's Data APIs.");`
`87`	`87`	`}`
	`88`	`+`
	`89`	`+ /**`
	`90`	`+ * @param string $message`
	`91`	`+ * @return self`
	`92`	`+ */`
	`93`	`+ public static function withMessage(string $message): self`
	`94`	`+ {`
	`95`	`+ return new static($message);`
	`96`	`+ }`
`88`	`97`	`}`