Skip to content

Commit b95642c

Browse files
Excludes OPTIONS requests in Graph permissions plugins (#807)
1 parent 66af639 commit b95642c

File tree

2 files changed

+9
-0
lines changed

2 files changed

+9
-0
lines changed

dev-proxy-plugins/RequestLogs/MinimalPermissionsGuidancePlugin.cs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,11 @@ private async Task AfterRecordingStop(object? sender, RecordingArgs e)
7171

7272
var methodAndUrlString = request.MessageLines.First();
7373
var methodAndUrl = GetMethodAndUrl(methodAndUrlString);
74+
if (methodAndUrl.method.Equals("OPTIONS", StringComparison.OrdinalIgnoreCase))
75+
{
76+
continue;
77+
}
78+
7479
var requestsFromBatch = Array.Empty<(string method, string url)>();
7580

7681
var uri = new Uri(methodAndUrl.url);

dev-proxy-plugins/RequestLogs/MinimalPermissionsPlugin.cs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,10 @@ private async Task AfterRecordingStop(object? sender, RecordingArgs e)
6262

6363
var methodAndUrlString = request.MessageLines.First();
6464
var methodAndUrl = GetMethodAndUrl(methodAndUrlString);
65+
if (methodAndUrl.method.Equals("OPTIONS", StringComparison.OrdinalIgnoreCase))
66+
{
67+
continue;
68+
}
6569

6670
var uri = new Uri(methodAndUrl.url);
6771
if (!ProxyUtils.IsGraphUrl(uri))

0 commit comments

Comments
 (0)