Excludes OPTIONS requests in Graph permissions plugins (#807)

microsoft · Jun 26, 2024 · b95642c · b95642c
1 parent 66af639
commit b95642c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/dev-proxy-plugins/RequestLogs/MinimalPermissionsGuidancePlugin.cs b/dev-proxy-plugins/RequestLogs/MinimalPermissionsGuidancePlugin.cs
@@ -71,6 +71,11 @@ private async Task AfterRecordingStop(object? sender, RecordingArgs e)
 
             var methodAndUrlString = request.MessageLines.First();
             var methodAndUrl = GetMethodAndUrl(methodAndUrlString);
+            if (methodAndUrl.method.Equals("OPTIONS", StringComparison.OrdinalIgnoreCase))
+            {
+                continue;
+            }
+
             var requestsFromBatch = Array.Empty<(string method, string url)>();
 
             var uri = new Uri(methodAndUrl.url);

diff --git a/dev-proxy-plugins/RequestLogs/MinimalPermissionsPlugin.cs b/dev-proxy-plugins/RequestLogs/MinimalPermissionsPlugin.cs
@@ -62,6 +62,10 @@ private async Task AfterRecordingStop(object? sender, RecordingArgs e)
 
             var methodAndUrlString = request.MessageLines.First();
             var methodAndUrl = GetMethodAndUrl(methodAndUrlString);
+            if (methodAndUrl.method.Equals("OPTIONS", StringComparison.OrdinalIgnoreCase))
+            {
+                continue;
+            }
 
             var uri = new Uri(methodAndUrl.url);
             if (!ProxyUtils.IsGraphUrl(uri))