-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add support for vaultUrl in azure keyvault plugin #60
Conversation
f349577
to
88a6fbb
Compare
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.
Looks good! I just have one small edit to the docs
Signed-off-by: Anat Balzam <13421451+anatbal@users.noreply.github.com>
Signed-off-by: Anat Balzam <13421451+anatbal@users.noreply.github.com>
Signed-off-by: Anat Balzam <13421451+anatbal@users.noreply.github.com>
Co-authored-by: Carolyn Van Slyck <me@carolynvanslyck.com> Signed-off-by: Anat Balzam <13421451+anatbal@users.noreply.github.com>
2816161
to
a37b6f2
Compare
@carolynvs Thanks! I fixed the docs with your suggestion |
Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>
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.
Thank you for the fix! 🎉
Your fix is now available in the v1.2.0 release: https://github.com/getporter/azure-plugins/releases/tag/v1.2.0 |
Currently, the Azure Key Vault plugin enforces that the URL of the vault must follow the format of "https://%s.vault.azure.net".
A new configuration parameter called "vaultURL" has been added in this pull request. This new parameter allows for the possibility of other suffixes to be used, thereby enabling support for multiple clouds, such as the government cloud.
It's worth noting that the "vault" parameter will still be available, but its value will be overridden by the "vaultURL" parameter if exists.