Skip to content

Commit d98a4ea

Browse files
authored
Suppress SM05137 in deserialization variable replacement settings (#3004)
## Why make this change? Silences CodeQL flag. ## What is this change? Adds the suppression language to the usage of `DefaultAzureCredential()` ## How was this tested? Against usual test suite, no real code change, just a comment. ## Sample Request(s) N/A
1 parent e83ccd8 commit d98a4ea

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Config/DeserializationVariableReplacementSettings.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ private static SecretClient CreateSecretClient(AzureKeyVaultOptions options)
259259
clientOptions.Retry.NetworkTimeout = TimeSpan.FromSeconds(options.RetryPolicy.NetworkTimeoutSeconds ?? AKVRetryPolicyOptions.DEFAULT_NETWORK_TIMEOUT_SECONDS);
260260
}
261261

262-
return new SecretClient(new Uri(options.Endpoint), new DefaultAzureCredential(), clientOptions);
262+
return new SecretClient(new Uri(options.Endpoint), new DefaultAzureCredential(), clientOptions); // CodeQL [SM05137] DefaultAzureCredential will use Managed Identity if available or fallback to default.
263263
}
264264

265265
private string? GetAkvVariable(string name)

0 commit comments

Comments
 (0)