Skip to content

Commit 63291d9

Browse files
committed
Evaluate excessivePermissions and properly initiate flag UsesMinimalPermissions
1 parent 7555b0c commit 63291d9

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

DevProxy.Plugins/Reporting/MinimalPermissionsGuidancePlugin.cs

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,11 @@ public override async Task AfterRecordingStopAsync(RecordingArgs e, Cancellation
103103
{
104104
var minimalPermissions = apiSpec.CheckMinimalPermissions(requests, Logger);
105105

106+
IEnumerable<string> excessivePermissions = [.. minimalPermissions.TokenPermissions
107+
.Except(Configuration.PermissionsToExclude ?? [])
108+
.Except(minimalPermissions.MinimalScopes)
109+
];
110+
106111
var result = new MinimalPermissionsGuidancePluginReportApiResult
107112
{
108113
ApiName = GetApiName(minimalPermissions.OperationsFromRequests.Any() ?
@@ -112,11 +117,8 @@ public override async Task AfterRecordingStopAsync(RecordingArgs e, Cancellation
112117
.Distinct()],
113118
TokenPermissions = [.. minimalPermissions.TokenPermissions.Distinct()],
114119
MinimalPermissions = minimalPermissions.MinimalScopes,
115-
ExcessivePermissions = [.. minimalPermissions.TokenPermissions
116-
.Except(Configuration.PermissionsToExclude ?? [])
117-
.Except(minimalPermissions.MinimalScopes)
118-
],
119-
UsesMinimalPermissions = !minimalPermissions.TokenPermissions.Except(minimalPermissions.MinimalScopes).Any()
120+
ExcessivePermissions = excessivePermissions,
121+
UsesMinimalPermissions = !excessivePermissions.Any()
120122
};
121123
results.Add(result);
122124

0 commit comments

Comments
 (0)