Skip to content

Commit 6c89a22

Browse files
Merge branch 'main' into 1052_add-scopes-to-ignore-in-MinimalPermissionsGuidancePlugin
2 parents 63291d9 + 80126c4 commit 6c89a22

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

.github/workflows/codeql-analysis.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ jobs:
5353

5454
# Initializes the CodeQL tools for scanning.
5555
- name: Initialize CodeQL
56-
uses: github/codeql-action/init@df559355d593797519d70b90fc8edd5db049e7a2 # v3.29.9
56+
uses: github/codeql-action/init@3c3833e0f8c1c83d449a7478aa59c036a9165498 # v3.29.11
5757
with:
5858
languages: ${{ matrix.language }}
5959
# If you wish to specify custom queries, you can do so here or in a config file.
@@ -81,4 +81,4 @@ jobs:
8181
run: dotnet build --no-restore -c Release
8282

8383
- name: Perform CodeQL Analysis
84-
uses: github/codeql-action/analyze@df559355d593797519d70b90fc8edd5db049e7a2 # v3.29.9
84+
uses: github/codeql-action/analyze@3c3833e0f8c1c83d449a7478aa59c036a9165498 # v3.29.11

DevProxy.Plugins/Mocking/AuthPlugin.cs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,9 @@ public override async Task InitializeAsync(InitArgs e, CancellationToken cancell
9292
{
9393
await base.InitializeAsync(e, cancellationToken);
9494

95+
// Disable by default to support early exits on configuration errors
96+
Enabled = false;
97+
9598
if (Configuration.Type == null)
9699
{
97100
Logger.LogError("Auth type is required");
@@ -153,7 +156,8 @@ public override async Task InitializeAsync(InitArgs e, CancellationToken cancell
153156
await SetupOpenIdConnectConfigurationAsync(Configuration.OAuth2.MetadataUrl);
154157
}
155158

156-
Enabled = false;
159+
// Enable the plugin after successful initialization
160+
Enabled = true;
157161
}
158162

159163
public override Task BeforeRequestAsync(ProxyRequestArgs e, CancellationToken cancellationToken)

0 commit comments

Comments
 (0)