libopenstorage / secrets

Openstorage support for Key Management Systems
Apache License 2.0
16 stars 16 forks source link

Support using certificates for Azure Key vault. #81

Open sp98 opened 7 months ago

sp98 commented 7 months ago

Is this a BUG REPORT or FEATURE REQUEST?: FEATURE REQUEST

What happened: As of now, the azure key vault client can be created using the ClientID and ClientSecret. The recommended way of creating the client is using a certificate.

What you expected to happen: Create azure key vault client using certificate. Use the NewServicePrincipalTokenFromCertificate function to generate the token using certificate.

How to reproduce it (as minimally and precisely as possible): N/A

Anything else we need to know?: N/A

Environment:

sp98 commented 7 months ago

@adityadani

I'm from the Rook team. I would like to take this up and work on it. Just wanted to check if you are still accepting contributions.

adityadani commented 7 months ago

@sp98 Yes feel free to send out a PR and we will review it. cc @dahuang-purestorage