Add a minor delay when fetching audit logs since the bot is too fast …

…between Discord sending the gateway event and updating the audit logs on server-side
OoLunar · Nov 1, 2024 · b635b5a · b635b5a
1 parent a882a3a
commit b635b5a
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 0 deletions.
diff --git a/src/Events/Handlers/BanAddedEventHandler.cs b/src/Events/Handlers/BanAddedEventHandler.cs
@@ -49,6 +49,9 @@ private static async ValueTask LogBanAsync(DiscordMember member)
             // Ensure all audit logs are the latest
             DateTimeOffset timestamp = DateTimeOffset.UtcNow.AddSeconds(-3);
 
+            // Wait 1 second so the audit logs can catch up
+            await Task.Delay(1000);
+
             // Figure out who unbanned the user
             await foreach (DiscordAuditLogEntry entry in member.Guild.GetAuditLogsAsync(100, null, DiscordAuditLogActionType.Ban))
             {

diff --git a/src/Events/Handlers/BanRemovedEventHandler.cs b/src/Events/Handlers/BanRemovedEventHandler.cs
@@ -48,6 +48,9 @@ private static async ValueTask LogUnbanAsync(DiscordMember member)
             // Ensure all audit logs are the latest
             DateTimeOffset timestamp = DateTimeOffset.UtcNow.AddSeconds(-3);
 
+            // Wait 1 second so the audit logs can catch up
+            await Task.Delay(1000);
+
             // Figure out who unbanned the user
             await foreach (DiscordAuditLogEntry entry in member.Guild.GetAuditLogsAsync(100, null, DiscordAuditLogActionType.Unban))
             {

diff --git a/src/Events/Handlers/GuildMemberRemovedEventHandler.cs b/src/Events/Handlers/GuildMemberRemovedEventHandler.cs
@@ -45,6 +45,9 @@ private static async ValueTask LogLeaveAsync(DiscordMember member)
 
             // Ensure all audit logs are the latest
             DateTimeOffset timestamp = DateTimeOffset.UtcNow.AddSeconds(-3);
+
+            // Wait 1 second so the audit logs can catch up
+            await Task.Delay(1000);
             await foreach (DiscordAuditLogEntry entry in member.Guild.GetAuditLogsAsync(100))
             {
                 if (LoggingEventHandlers.TryFilterAuditLogEntry(entry, DiscordAuditLogActionType.Ban, timestamp, out DiscordAuditLogBanEntry? banEntry) && banEntry.Target.Id == member.Id)

diff --git a/src/Events/Handlers/GuildMemberUpdatedEventHandler.cs b/src/Events/Handlers/GuildMemberUpdatedEventHandler.cs
@@ -86,6 +86,9 @@ private static async ValueTask LogMemberUpdateAsync(GuildMemberUpdatedEventArgs
 
                 // Figure out who muted the user
                 DateTimeOffset timestamp = DateTimeOffset.UtcNow.AddSeconds(-3);
+
+                // Wait 1 second so the audit logs can catch up
+                await Task.Delay(1000);
                 await foreach (DiscordAuditLogEntry entry in eventArgs.Guild.GetAuditLogsAsync(100, null, DiscordAuditLogActionType.MemberUpdate))
                 {
                     if (LoggingEventHandlers.TryFilterAuditLogEntry(entry, DiscordAuditLogActionType.MemberUpdate, timestamp, out DiscordAuditLogMemberUpdateEntry? updateEntry) && updateEntry.Target.Id == eventArgs.Member.Id)