SecretManagement icon indicating copy to clipboard operation
SecretManagement copied to clipboard

Unregister-SecretVault should be ConfirmImpact "High"

Open JustinGrote opened this issue 5 years ago • 1 comments

It is common a user may be pipelining unregistrations, so commands like Get-SecretVault | Unregister-SecretVault which can unregister a ton of vaults should not be allowed to happen without at least some sort of confirmation

JustinGrote avatar Jan 16 '21 04:01 JustinGrote

Thanks @JustinGrote we made this choice to follow the pattern of other Unregister cmdlets in PowerShell (like scheduled jobs), and because it does not necessarily remove the underlying vault, however we will leave this open for discussion

SydneyhSmith avatar Jan 19 '21 22:01 SydneyhSmith