From 9136d634b07b3e32950405966db16148875136b4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mario=20Alejandro=20Montoya=20Corte=CC=81s?=
 <mamcx@elmalabarista.com>
Date: Wed, 18 Sep 2024 11:57:17 -0500
Subject: [PATCH] Fixes showing server errors on sql calls

---
 crates/cli/src/errors.rs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/crates/cli/src/errors.rs b/crates/cli/src/errors.rs
index c2303f1de4..4b7dd68d37 100644
--- a/crates/cli/src/errors.rs
+++ b/crates/cli/src/errors.rs
@@ -25,7 +25,8 @@ pub async fn error_for_status(response: Response) -> Result<Response, CliError>
     if let Some(kind) = status
         .is_client_error()
         .then_some(RequestSource::Client)
-        .or_else(|| status.is_client_error().then_some(RequestSource::Server))
+        // Anything that is not a success is an error for the client, even a redirect that is not followed.
+        .or_else(|| (!status.is_success()).then_some(RequestSource::Server))
     {
         let msg = response.text().await?;
         return Err(CliError::Request { kind, msg, status });