-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature - track Azure Key Vault dependency #207
Feature - track Azure Key Vault dependency #207
Conversation
A new preview package for You can pull it locally via the CLI: PM> Install-Package Arcus.Security.All -Version 20201112.0.0-PR-207 -Source https://www.myget.org/F/arcus/api/v3/index.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but made some suggestions.
Would you mind posting the result in application insights with these changes please?
src/Arcus.Security.Providers.AzureKeyVault/Configuration/KeyVaultOptions.cs
Show resolved
Hide resolved
src/Arcus.Security.Providers.AzureKeyVault/KeyVaultSecretProvider.cs
Outdated
Show resolved
Hide resolved
We could do that, I'll have to set up a Serilog logger so we can use our sink. |
Opt-in to track the Azure Key Vault dependency, while maintaining backwards compatibility towards the preview v1.3.0 (which didn't have mutate secret names).
Closes #182
Relates to #177