From 09dff173a6401cb62a9204a0d6fb52fe93cd0568 Mon Sep 17 00:00:00 2001
From: Taylor Baldwin <taylorbaldwin@gmail.com>
Date: Fri, 17 Jan 2025 10:41:57 -0500
Subject: [PATCH] only log plane client errors for 5xx-level responses

---
 common/src/lib.rs | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/common/src/lib.rs b/common/src/lib.rs
index 1f56398ce..1177c815f 100644
--- 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 {