migrations/2025-07-22-195121_add_ban/down.sql

@@ -1,2 +0,0 @@
--- This file should undo anything in `up.sql`
-DROP TABLE guild_bans;

migrations/2025-07-22-195121_add_ban/up.sql

@@ -1,8 +0,0 @@
--- Your SQL goes here
-CREATE TABLE guild_bans (
-	guild_uuid uuid NOT NULL REFERENCES guilds(uuid) ON DELETE CASCADE,
-	user_uuid uuid NOT NULL REFERENCES users(uuid),
-	reason VARCHAR(200) DEFAULT NULL,
-	banned_since TIMESTAMPTZ NOT NULL DEFAULT NOW(),
-	PRIMARY KEY (user_uuid, guild_uuid)
-);

src/api/v1/guilds/uuid/bans.rs

@@ -1,57 +0,0 @@
-use std::sync::Arc;
-
-use axum::{
-    Extension, Json,
-    extract::{Path, State},
-    http::StatusCode,
-    response::IntoResponse,
-};
-use uuid::Uuid;
-
-use crate::{
-    AppState,
-    api::v1::auth::CurrentUser,
-    error::Error,
-    objects::{GuildBan, Member, Permissions},
-    utils::global_checks,
-};
-
-pub async fn get(
-    State(app_state): State<Arc<AppState>>,
-    Path(guild_uuid): Path<Uuid>,
-    Extension(CurrentUser(uuid)): Extension<CurrentUser<Uuid>>,
-) -> Result<impl IntoResponse, Error> {
-    global_checks(&app_state, uuid).await?;
-
-    let mut conn = app_state.pool.get().await?;
-
-    let caller = Member::check_membership(&mut conn, uuid, guild_uuid).await?;
-    caller
-        .check_permission(&app_state, Permissions::BanMember)
-        .await?;
-
-    let all_guild_bans = GuildBan::fetch_all(&mut conn, guild_uuid).await?;
-
-    Ok((StatusCode::OK, Json(all_guild_bans)))
-}
-
-pub async fn unban(
-    State(app_state): State<Arc<AppState>>,
-    Path((guild_uuid, user_uuid)): Path<(Uuid, Uuid)>,
-    Extension(CurrentUser(uuid)): Extension<CurrentUser<Uuid>>,
-) -> Result<impl IntoResponse, Error> {
-    global_checks(&app_state, uuid).await?;
-
-    let mut conn = app_state.pool.get().await?;
-
-    let caller = Member::check_membership(&mut conn, uuid, guild_uuid).await?;
-    caller
-        .check_permission(&app_state, Permissions::BanMember)
-        .await?;
-
-    let ban = GuildBan::fetch_one(&mut conn, guild_uuid, user_uuid).await?;
-
-    ban.unban(&mut conn).await?;
-
-    Ok(StatusCode::OK)
-}

src/api/v1/guilds/uuid/mod.rs

@@ -7,12 +7,11 @@ use axum::{
     extract::{Multipart, Path, State},
     http::StatusCode,
     response::IntoResponse,
-    routing::{delete, get, patch, post},
+    routing::{get, patch, post},
 };
 use bytes::Bytes;
 use uuid::Uuid;
 
-mod bans;
 mod channels;
 mod invites;
 mod members;
@@ -43,9 +42,6 @@ pub fn router() -> Router<Arc<AppState>> {
         .route("/invites", post(invites::create))
         // Members
         .route("/members", get(members::get))
-        // Bans
-        .route("/bans", get(bans::get))
-        .route("/bans/{uuid}", delete(bans::unban))
 }
 
 /// `GET /api/v1/guilds/{uuid}` DESCRIPTION

src/api/v1/members/mod.rs

@@ -2,7 +2,7 @@ use std::sync::Arc;
 
 use axum::{
     Router,
-    routing::{delete, get, post},
+    routing::{delete, get},
 };
 
 use crate::AppState;
@@ -13,5 +13,4 @@ pub fn router() -> Router<Arc<AppState>> {
     Router::new()
         .route("/{uuid}", get(uuid::get))
         .route("/{uuid}", delete(uuid::delete))
-        .route("/{uuid}/ban", post(uuid::ban::post))
 }

src/api/v1/members/uuid/ban.rs

@@ -1,47 +0,0 @@
-use std::sync::Arc;
-
-use axum::{
-    Extension,
-    extract::{Json, Path, State},
-    http::StatusCode,
-    response::IntoResponse,
-};
-use serde::Deserialize;
-
-use crate::{
-    AppState,
-    api::v1::auth::CurrentUser,
-    error::Error,
-    objects::{Member, Permissions},
-    utils::global_checks,
-};
-
-use uuid::Uuid;
-
-#[derive(Deserialize)]
-pub struct RequstBody {
-    reason: String,
-}
-
-pub async fn post(
-    State(app_state): State<Arc<AppState>>,
-    Path(member_uuid): Path<Uuid>,
-    Extension(CurrentUser(uuid)): Extension<CurrentUser<Uuid>>,
-    Json(payload): Json<RequstBody>,
-) -> Result<impl IntoResponse, Error> {
-    global_checks(&app_state, uuid).await?;
-
-    let mut conn = app_state.pool.get().await?;
-
-    let member = Member::fetch_one_with_member(&app_state, None, member_uuid).await?;
-
-    let caller = Member::check_membership(&mut conn, uuid, member.guild_uuid).await?;
-
-    caller
-        .check_permission(&app_state, Permissions::BanMember)
-        .await?;
-
-    member.ban(&mut conn, &payload.reason).await?;
-
-    Ok(StatusCode::OK)
-}

src/api/v1/members/uuid/mod.rs

@@ -1,7 +1,5 @@
 //! `/api/v1/members/{uuid}` Member specific endpoints
 
-pub mod ban;
-
 use std::sync::Arc;
 
 use crate::{
@@ -31,7 +29,7 @@ pub async fn get(
 
     let me = Me::get(&mut conn, uuid).await?;
 
-    let member = Member::fetch_one_with_member(&app_state, Some(&me), member_uuid).await?;
+    let member = Member::fetch_one_with_member(&app_state, &me, member_uuid).await?;
     Member::check_membership(&mut conn, uuid, member.guild_uuid).await?;
 
     Ok((StatusCode::OK, Json(member)))
@@ -48,7 +46,7 @@ pub async fn delete(
 
     let me = Me::get(&mut conn, uuid).await?;
 
-    let member = Member::fetch_one_with_member(&app_state, Some(&me), member_uuid).await?;
+    let member = Member::fetch_one_with_member(&app_state, &me, member_uuid).await?;
 
     let deleter = Member::check_membership(&mut conn, uuid, member.guild_uuid).await?;
 

src/objects/bans.rs

@@ -1,57 +0,0 @@
-use diesel::{ExpressionMethods, QueryDsl, Queryable, Selectable, SelectableHelper};
-use serde::{Deserialize, Serialize};
-use uuid::Uuid;
-
-use diesel_async::RunQueryDsl;
-
-use crate::{Conn, error::Error, objects::load_or_empty, schema::guild_bans};
-
-#[derive(Selectable, Queryable, Serialize, Deserialize)]
-#[diesel(table_name = guild_bans)]
-#[diesel(check_for_backend(diesel::pg::Pg))]
-pub struct GuildBan {
-    pub guild_uuid: Uuid,
-    pub user_uuid: Uuid,
-    pub reason: Option<String>,
-    pub banned_since: chrono::DateTime<chrono::Utc>,
-}
-
-impl GuildBan {
-    pub async fn fetch_one(
-        conn: &mut Conn,
-        guild_uuid: Uuid,
-        user_uuid: Uuid,
-    ) -> Result<GuildBan, Error> {
-        use guild_bans::dsl;
-        let guild_ban = dsl::guild_bans
-            .filter(dsl::guild_uuid.eq(guild_uuid))
-            .filter(dsl::user_uuid.eq(user_uuid))
-            .select(GuildBan::as_select())
-            .get_result(conn)
-            .await?;
-
-        Ok(guild_ban)
-    }
-
-    pub async fn fetch_all(conn: &mut Conn, guild_uuid: Uuid) -> Result<Vec<Self>, Error> {
-        use guild_bans::dsl;
-        let all_guild_bans = load_or_empty(
-            dsl::guild_bans
-                .filter(dsl::guild_uuid.eq(guild_uuid))
-                .load(conn)
-                .await,
-        )?;
-
-        Ok(all_guild_bans)
-    }
-
-    pub async fn unban(self, conn: &mut Conn) -> Result<(), Error> {
-        use guild_bans::dsl;
-        diesel::delete(guild_bans::table)
-            .filter(dsl::guild_uuid.eq(self.guild_uuid))
-            .filter(dsl::user_uuid.eq(self.user_uuid))
-            .execute(conn)
-            .await?;
-        Ok(())
-    }
-}

src/objects/member.rs

@@ -9,8 +9,8 @@ use uuid::Uuid;
 use crate::{
     AppState, Conn,
     error::Error,
-    objects::{GuildBan, Me, Permissions, Role},
+    objects::{Me, Permissions, Role},
-    schema::{guild_bans, guild_members},
+    schema::guild_members,
 };
 
 use super::{User, load_or_empty};
@@ -122,7 +122,7 @@ impl Member {
 
     pub async fn fetch_one_with_member(
         app_state: &AppState,
-        me: Option<&Me>,
+        me: &Me,
         uuid: Uuid,
     ) -> Result<Self, Error> {
         let mut conn = app_state.pool.get().await?;
@@ -134,7 +134,7 @@ impl Member {
             .get_result(&mut conn)
             .await?;
 
-        member.build(app_state, me).await
+        member.build(app_state, Some(me)).await
     }
 
     pub async fn fetch_all(
@@ -169,13 +169,6 @@ impl Member {
     ) -> Result<Self, Error> {
         let mut conn = app_state.pool.get().await?;
 
-        let banned = GuildBan::fetch_one(&mut conn, guild_uuid, user_uuid).await;
-        match banned {
-            Ok(_) => Err(Error::Forbidden("User banned".to_string())),
-            Err(Error::SqlError(diesel::result::Error::NotFound)) => Ok(()),
-            Err(e) => Err(e),
-        }?;
-
         let member_uuid = Uuid::now_v7();
 
         let member = MemberBuilder {
@@ -205,24 +198,4 @@ impl Member {
 
         Ok(())
     }
-
-    pub async fn ban(self, conn: &mut Conn, reason: &String) -> Result<(), Error> {
-        if self.is_owner {
-            return Err(Error::Forbidden("Can not ban owner".to_string()));
-        }
-
-        use guild_bans::dsl;
-        insert_into(guild_bans::table)
-            .values((
-                dsl::guild_uuid.eq(self.guild_uuid),
-                dsl::user_uuid.eq(self.user_uuid),
-                dsl::reason.eq(reason),
-            ))
-            .execute(conn)
-            .await?;
-
-        self.delete(conn).await?;
-
-        Ok(())
-    }
 }

src/objects/mod.rs

@@ -7,7 +7,6 @@ use log::debug;
 use serde::Deserialize;
 use uuid::Uuid;
 
-mod bans;
 mod channel;
 mod email_token;
 mod friends;
@@ -20,7 +19,6 @@ mod password_reset_token;
 mod role;
 mod user;
 
-pub use bans::GuildBan;
 pub use channel::Channel;
 pub use email_token::EmailToken;
 pub use friends::Friend;

src/objects/role.rs

@@ -176,8 +176,6 @@ pub enum Permissions {
     ManageGuild = 32,
     /// Lets users change member settings (nickname, etc)
     ManageMember = 64,
-    /// Lets users ban members
-    BanMember = 128,
     /// Lets users kick members
     KickMember = 256,
 }
@@ -192,7 +190,6 @@ impl Permissions {
             Self::ManageInvite,
             Self::ManageGuild,
             Self::ManageMember,
-            Self::BanMember,
             Self::KickMember,
         ];
 

src/schema.rs

@@ -47,16 +47,6 @@ diesel::table! {
     }
 }
 
-diesel::table! {
-    guild_bans (user_uuid, guild_uuid) {
-        guild_uuid -> Uuid,
-        user_uuid -> Uuid,
-        #[max_length = 200]
-        reason -> Nullable<Varchar>,
-        banned_since -> Timestamptz,
-    }
-}
-
 diesel::table! {
     guild_members (uuid) {
         uuid -> Uuid,
@@ -164,8 +154,6 @@ diesel::joinable!(access_tokens -> refresh_tokens (refresh_token));
 diesel::joinable!(access_tokens -> users (uuid));
 diesel::joinable!(channel_permissions -> channels (channel_uuid));
 diesel::joinable!(channels -> guilds (guild_uuid));
-diesel::joinable!(guild_bans -> guilds (guild_uuid));
-diesel::joinable!(guild_bans -> users (user_uuid));
 diesel::joinable!(guild_members -> guilds (guild_uuid));
 diesel::joinable!(guild_members -> users (user_uuid));
 diesel::joinable!(instance_permissions -> users (uuid));
@@ -183,7 +171,6 @@ diesel::allow_tables_to_appear_in_same_query!(
     channels,
     friend_requests,
     friends,
-    guild_bans,
     guild_members,
     guilds,
     instance_permissions,