Skip to content

Comments

This adds the ability to add vaults from the same tenant, by subscription id, rg name and vault name, since traversing the dep tree can often not show all the vaults the user has access to.#116

Merged
cricketthomas merged 1 commit intomasterfrom
107-add-ability-to-add-vault-via-uri
Mar 26, 2025
Merged

This adds the ability to add vaults from the same tenant, by subscription id, rg name and vault name, since traversing the dep tree can often not show all the vaults the user has access to.#116
cricketthomas merged 1 commit intomasterfrom
107-add-ability-to-add-vault-via-uri

Conversation

@cricketthomas
Copy link
Owner

@cricketthomas cricketthomas commented Mar 26, 2025

This adds the ability to add vaults from the same tenant, by subscription id, rg name and vault name, since traversing the dep tree can often not show all the vaults the user has access to. This will also pin the vault as a courtesy. This serves as a solution to #107 until more features in regards to opening vaults, and one off kv values can be implemented.

The user will still need access to be able to get/list all secrets etc in the vault.

image
image
Error handling:
image

…tion id, rg name and vault name, since traversing the dep tree can often not show all the vaults the user has access to.
@cricketthomas cricketthomas requested a review from GSerjo March 26, 2025 12:15
@cricketthomas cricketthomas merged commit f833fc2 into master Mar 26, 2025
1 of 2 checks passed
@cricketthomas cricketthomas deleted the 107-add-ability-to-add-vault-via-uri branch June 9, 2025 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants