weiguoqiang/cert-manager
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

venafi: ResetCertificate wasn't working

Browse Source 
Signed-off-by: Maël Valais <mael@vls.dev>
This commit is contained in:
Maël Valais 2023-10-06 16:24:15 +02:00
parent d8035e668a
commit d1d92b6398
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
pkg/issuer/venafi/client/request.go
View File

@ -66,9 +66,12 @@ func (v *Venafi) RequestCertificate(csrPEM []byte, duration time.Duration, custo
//
// Note that resetting won't affect the existing certificate if one was
// already issued.
tppConnector, isTPP := v.vcertClient.(*tpp.Connector)
if isTPP {
err := tppConnector.ResetCertificate(vreq, false)
if v.tppClient != nil {
// We can't use the instrumented v.vcertClient because its concrete
// value is `instrumentedConnector`, which doesn't give access to the
// *tpp.Connector it wraps. Also, `instrumentedConnector` doesn't
// support `ResetCertificate`.
err := v.tppClient.ResetCertificate(vreq, false)
notFoundErr := &tpp.ErrCertNotFound{}
if err != nil && !errors.As(err, &notFoundErr) {
return "", err