Shortcut file-added earlier (#3596)

microsoft · Oct 24, 2023 · ca3c503 · ca3c503
1 parent 9ba004f
commit ca3c503
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/ApiService/ApiService/Functions/QueueFileChanges.cs b/src/ApiService/ApiService/Functions/QueueFileChanges.cs
@@ -76,6 +76,11 @@ private async Async.Task<OneFuzzResultVoid> FileAdded(ResourceIdentifier storage
         var container = Container.Parse(parts[0]);
         var path = string.Join('/', parts.Skip(1));
 
+        // We don't want to store file added events for the events container because that causes an infinite loop
+        if (container == WellKnownContainers.Events) {
+            return Result.Ok();
+        }
+
         _log.LogInformation("file added : {Container} - {Path}", container.String, path);
 
         var (_, result) = await (

diff --git a/src/ApiService/ApiService/onefuzzlib/NotificationOperations.cs b/src/ApiService/ApiService/onefuzzlib/NotificationOperations.cs
@@ -22,11 +22,6 @@ public NotificationOperations(ILogger<NotificationOperations> log, IOnefuzzConte
 
     }
     public async Async.Task<OneFuzzResultVoid> NewFiles(Container container, string filename) {
-        // We don't want to store file added events for the events container because that causes an infinite loop
-        if (container == WellKnownContainers.Events) {
-            return Result.Ok();
-        }
-
         var result = OneFuzzResultVoid.Ok;
         var notifications = GetNotifications(container);
         var hasNotifications = await notifications.AnyAsync();