chore: Handle failed banning properly

Signed-off-by: Awiteb <a@4rs.nl>

src/error.rs

@@ -14,6 +14,8 @@
 // You should have received a copy of the GNU Affero General Public License
 // along with this program. If not, see <https://gnu.org/licenses/agpl.txt>.
 
+use reqwest::StatusCode;
+
 use crate::config::{CONFIG_PATH_ENV, DEFAULT_CONFIG_PATH};
 
 /// Result of the guard
@@ -39,4 +41,7 @@ pub enum GuardError {
     /// Faild to deserialize the config file
     #[error("Failed to deserialize the config: {0}")]
     FaildDeserializeConfig(#[from] toml::de::Error),
+    /// Failed to ban the user
+    #[error("Failed to ban the user, status code: {0}")]
+    FailedToBan(StatusCode),
 }

src/forgejo_api/ban_user.rs

@@ -16,7 +16,7 @@
 
 use reqwest::Method;
 
-use crate::error::GuardResult;
+use crate::error::{GuardError, GuardResult};
 
 /// Ban a user from the instance, purging their data.
 pub async fn ban_user(
@@ -34,7 +34,10 @@ pub async fn ban_user(
         ))
         .await?;
     tracing::debug!("Ban user response: {:?}", &res);
-    tracing::debug!("Body: {}", res.text().await.unwrap_or_default());
+
+    if !res.status().is_success() {
+        return Err(GuardError::FailedToBan(res.status()));
+    }
 
     Ok(())
 }