only log plane client errors for 5xx-level responses (#859)

jamsocket · Jan 17, 2025 · 04c9f12 · 04c9f12
1 parent a9d5ccf
commit 04c9f12
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/common/src/lib.rs b/common/src/lib.rs
@@ -217,8 +217,16 @@ async fn get_response<T: DeserializeOwned>(response: Response) -> Result<T, Plan
         Ok(response.json::<T>().await?)
     } else {
         let url = response.url().to_string();
-        tracing::error!(?url, "Got error response from API server.");
         let status = response.status();
+        if status.is_server_error() {
+            tracing::error!(?url, ?status, "Got 5xx response from Plane API server.");
+        } else {
+            tracing::warn!(
+                ?url,
+                ?status,
+                "Got unsuccessful response from Plane API server."
+            );
+        }
         if let Ok(api_error) = response.json::<ApiError>().await {
             Err(PlaneClientError::PlaneError(api_error, status))
         } else {