shibayan / keyvault-acmebot

Automated ACME SSL/TLS certificates issuer for Azure Key Vault (App Service / Container Apps / App Gateway / Front Door / CDN / others)
Apache License 2.0
908 stars 233 forks source link

Confirmation that certificate will be replaced #565

Open davidonus opened 1 year ago

davidonus commented 1 year ago

Great project and have been using it at my company for managing let's encrypt certs.

One thing I have noticed is that if you amass several certificates, it is open to human error in replacing the certificates.

Unless you manually check through all your current certs you can replace one if you use the same name.

Is there scope to add a check on the add function to check if that name already exists and ask for confirmation that you are happy for it to be replaced?

shibayan commented 1 year ago

Good idea. It is not the desired behavior to have certificates overwritten, so it would be better to be able to issue a warning.