Azure / azure-sdk-for-js

This repository is for active development of the Azure SDK for JavaScript (NodeJS & Browser). For consumers of the SDK we recommend visiting our public developer docs at https://docs.microsoft.com/javascript/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-js.
MIT License
2.03k stars 1.19k forks source link

List Key Vault Certificates #30516

Closed akumar-99 closed 1 month ago

akumar-99 commented 1 month ago

Is your feature request related to a problem? Please describe. Package Name - @azure/arm-keyvault We have ways to list keys and secrets in a keyvault using keyVaultClient.keys.list and keyVaultClient.secrets.list respectively, but we do not have a way to do the same for certificates in a keyvault.

Describe the solution you'd like Please add a similar object and function to list certificates and their configurations stored in a keyvault in the @azure/arm-keyvault package.

github-actions[bot] commented 1 month ago

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

kazrael2119 commented 1 month ago

will check

kazrael2119 commented 1 month ago

sdk is generated from swagger, and for the keyVault mgmt-plane sdk, it doesn't support the api for certificates https://github.com/Azure/azure-rest-api-specs/tree/main/specification/keyvault/resource-manager/Microsoft.KeyVault/stable/2023-07-01. But it supports this api in data-plane:https://github.com/Azure/azure-rest-api-specs/blob/main/specification/keyvault/data-plane/Microsoft.KeyVault/stable/7.5/certificates.json, and we have a data-plane package for certificates https://www.npmjs.com/package/@azure/keyvault-certificates here.

qiaozha commented 1 month ago

@jlichwa @RandalliLama @schaabs please follow up with this issue. thanks.

timovv commented 1 month ago

Hello @akumar-99, since this operation is not available in the management plane REST API, I would recommend using the data plane SDK instead. CertificateClient in @azure/keyvault-certificates provides a listPropertiesOfCertificates function which may be suitable for your use case.

If using the ARM SDK is a requirement, then this would require changes to the REST API specification, which we cannot assist with here in the JavaScript SDK repo.

github-actions[bot] commented 1 month ago

Hi @akumar-99. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text "/unresolve" to remove the "issue-addressed" label and continue the conversation.

github-actions[bot] commented 1 month ago

Hi @akumar-99, since you haven’t asked that we /unresolve the issue, we’ll close this out. If you believe further discussion is needed, please add a comment /unresolve to reopen the issue.