properly catch, print and log errors when banning a user with .ban or…

… .banall
Manechat · Nov 7, 2023 · 235b6f0 · 235b6f0
1 parent d83e881
commit 235b6f0
Showing 1 changed file with 18 additions and 1 deletion.
diff --git a/Izzy-Moonbot/Modules/ModCoreModule.cs b/Izzy-Moonbot/Modules/ModCoreModule.cs
@@ -227,7 +227,24 @@ await Context.Channel.SendMessageAsync(
         if (!hasExistingBan)
         {
             // No ban exists, very serious Izzy time.
-            await Context.Guild.AddBanAsync(userId, pruneDays: 0, reason: reason );
+            try
+            {
+                await Context.Guild.AddBanAsync(userId, pruneDays: 0, reason: reason );
+            }
+            catch (Exception ex)
+            {
+                _logger.Log(
+                    $"AddBanAsync() threw an exception when trying to ban userId {userId}\n" +
+                    $"Type: {ex.GetType().Name}\n" +
+                    $"Message: {ex.Message}\n" +
+                    $"Job: {job.ToFileString()}\n" +
+                    $"Stack Trace: {ex.StackTrace}");
+
+                await Context.Channel.SendMessageAsync(
+                    $":warning: Failed to ban userId {userId}\n" +
+                    $"Error was: [{ex.GetType().Name}] {ex.Message}\n" +
+                    $"(Check Izzy's logs for a full stack trace)");
+            }
 
             if (time != null)
             {