Return an error if the reason the user provided exceeded the limit

2 files changed, 14 insertions, 2 deletions

diff --git a/src/model/guild/guild_id.rs b/src/model/guild/guild_id.rs
index c2cfc3d..c707413 100644
--- a/src/model/guild/guild_id.rs
+++ b/src/model/guild/guild_id.rs
@@ -53,7 +53,13 @@ impl GuildId {
             return Err(Error::Model(ModelError::DeleteMessageDaysAmount(dmd)));
         }
 
-        http::ban_user(self.0, user.into().0, dmd, &*ban_options.reason())
+        let reason = ban_options.reason();
+
+        if reason.len() > 512 {
+            return Err(Error::ExceededLimit);
+        }
+
+        http::ban_user(self.0, user.into().0, dmd, &*reason)
     }
 
     /// Gets a list of the guild's bans.
diff --git a/src/model/guild/member.rs b/src/model/guild/member.rs
index b128887..053360d 100644
--- a/src/model/guild/member.rs
+++ b/src/model/guild/member.rs
@@ -146,7 +146,13 @@ impl Member {
     /// [Ban Members]: permissions/constant.BAN_MEMBERS.html
     #[cfg(feature="cache")]
     pub fn ban<BO: BanOptions>(&self, ban_options: BO) -> Result<()> {
-        http::ban_user(self.guild_id.0, self.user.read().unwrap().id.0, ban_options.dmd(), &*ban_options.reason())
+        let reason = ban_options.reason();
+        
+        if reason.len() > 512 {
+            return Err(Error::ExceededLimit);
+        }
+
+        http::ban_user(self.guild_id.0, self.user.read().unwrap().id.0, ban_options.dmd(), &*reason)
     }
 
     /// Determines the member's colour.