possibly fix webhook stuff

fmbot-discord · Sep 23, 2024 · 02ab0b5 · 02ab0b5
1 parent aa784a8
commit 02ab0b5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/src/FMBot.Bot/Services/Guild/WebhookService.cs b/src/FMBot.Bot/Services/Guild/WebhookService.cs
@@ -1,5 +1,5 @@
 using System;
-using System.Diagnostics.Eventing.Reader;
+using System.Collections.Generic;
 using System.IO;
 using System.Linq;
 using System.Net.Http;
@@ -97,10 +97,10 @@ public async Task<Webhook> CreateWebhook(ICommandContext context, int guildId)
 
     public async Task<bool> TestWebhook(Webhook webhook, string text)
     {
+        var webhookClient = new DiscordWebhookClient(webhook.DiscordWebhookId, webhook.Token);
+
         try
         {
-            var webhookClient = new DiscordWebhookClient(webhook.DiscordWebhookId, webhook.Token);
-
             await webhookClient.SendMessageAsync(text, threadId: webhook.DiscordThreadId);
 
             return true;
@@ -123,6 +123,10 @@ public async Task<bool> TestWebhook(Webhook webhook, string text)
 
             return false;
         }
+        finally
+        {
+            webhookClient.Dispose();
+        }
     }
 
     public async Task SendFeaturedWebhooks(FeaturedLog featured)
@@ -136,10 +140,14 @@ public async Task SendFeaturedWebhooks(FeaturedLog featured)
             .AsQueryable()
             .ToListAsync();
 
+        var tasks = new List<Task>();
         foreach (var webhook in webhooks)
         {
-            await SendWebhookEmbed(webhook, embed, featured.UserId);
+            tasks.Add(SendWebhookEmbed(webhook, embed, featured.UserId));
         }
+
+        Log.Information("SendFeaturedWebhooks: Sending {webhookCount} featured webhooks", webhooks.Count);
+        await Task.WhenAll(tasks);
     }
 
     public async Task SendFeaturedPreview(FeaturedLog featured, string webhook)

diff --git a/src/FMBot.Bot/TextCommands/Guild/WebhookCommands.cs b/src/FMBot.Bot/TextCommands/Guild/WebhookCommands.cs
@@ -76,8 +76,6 @@ await ReplyAsync(
             return;
         }
 
-        var type = socketCommandContext.Channel.GetChannelType();
-
         var createdWebhook = await this._webhookService.CreateWebhook(this.Context, guild.GuildId);
 
         await this._webhookService.TestWebhook(createdWebhook, "If you see this message the webhook has been successfully added!\n\n" +