KeyPermissions is missing a value

[CYoderDev]

azure-sdk-for-js/sdk/keyvault/arm-keyvault/src/models/index.ts

Line 613 in f21b80d

export type KeyPermissions = 'encrypt' | 'decrypt' | 'wrapKey' | 'unwrapKey' | 'sign' | 'verify' | 'get' | 'list' | 'create' | 'update' | 'import' | 'delete' | 'backup' | 'restore' | 'recover' | 'purge';

The KeyPermissions, SecretPermissions, and CertificatePermissions types are missing the value all which is a possible value of access policy permissions when getting the key vault using the api and is supported by ARM template deployments.

Here is an example of an permissions of a access policy on a key vault that is returned by the api:

Permissions to Keys                        : {all}
Permissions to Secrets                     : {all}
Permissions to Certificates                : {all}

[sadasant]

Hello, @CYoderDev

Thank you for reaching out to us!

This issue will require further investigation, but so far I can say that:

• arm-keyvault hasn't been updated since November 2019.
• Not even the Azure SDK for .Net has this "all" value: https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/keyvault/Microsoft.Azure.Management.KeyVault/src/Generated/Models/KeyPermissions.cs#L11

I'll come back with a better response later today, or perhaps tomrorow.

[sadasant]

@CYoderDev Update:

The "all" value as a permission is not present in Azure's public API for Key Vault: https://github.com/Azure/azure-rest-api-specs/blob/a7ca142c3f0f290fdda778f26f4f1fab65f71cf9/specification/keyvault/resource-manager/Microsoft.KeyVault/stable/2019-09-01/keyvault.json#L979-L1059

I wonder what are we missing! I'm still in communications with my team. I'll report back as soon as I have an update.

[ghost]

Hi, we're sending this friendly reminder because we haven't heard back from you in a while. We need more information about this issue to help address it. Please be sure to give us your input within the next 7 days. If we don't hear back from you within 14 days of this comment the issue will be automatically closed. Thank you!

[ghost]

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @RandalliLama, @schaabs, @jlichwa.

[sadasant]

I've switched this issue to be a service-level issue. Hopefully we'll get more answers this way.

[jlichwa]

Linked rest api PR - Azure/azure-rest-api-specs#11042

[sadasant]

@jlichwa that looks cool! Thank you!

[sebansal]

Closing issue as its been addressed.