fix: do not wrap nil provisioners

Author: azazeal

authority/authority.go

@@ -968,16 +968,20 @@ func (a *Authority) startCRLGenerator() error {
 func (a *Authority) incrProvisionerCounter(prov *provisioner.Interface, err *error, count func(Meter, string, bool)) {
 	var name string
 	if p := *prov; p != nil {
-		// TODO(@azazeal): ???
+		name = p.GetName()
 
-		switch x := p.(type) {
-		case *wrappedProvisioner:
-			if x.Interface != nil {
+		/*
+			// TODO(@azazeal): ???
+
+			switch x := p.(type) {
+			case *wrappedProvisioner:
+				if x.Interface != nil {
+					name = x.GetName()
+				}
+			default:
 				name = x.GetName()
 			}
-		default:
-			name = x.GetName()
-		}
+		*/
 	}
 
 	count(a.meter, name, *err == nil)
@@ -986,16 +990,20 @@ func (a *Authority) incrProvisionerCounter(prov *provisioner.Interface, err *err
 func (a *Authority) incrWebhookCounter(prov provisioner.Interface, err error, count func(Meter, string, bool)) {
 	var name string
 	if prov != nil {
-		// TODO(@azazeal): ???
+		name = prov.GetName()
 
-		switch x := prov.(type) {
-		case *wrappedProvisioner:
-			if x.Interface != nil {
+		/*
+			// TODO(@azazeal): ???
+
+			switch x := prov.(type) {
+			case *wrappedProvisioner:
+				if x.Interface != nil {
+					name = x.GetName()
+				}
+			default:
 				name = x.GetName()
 			}
-		default:
-			name = x.GetName()
-		}
+		*/
 	}
 
 	count(a.meter, name, err == nil)

authority/tls.go

@@ -310,8 +310,10 @@ func (a *Authority) Sign(csr *x509.CertificateRequest, signOpts provisioner.Sign
 
 	cert = append([]*x509.Certificate{resp.Certificate}, resp.CertificateChain...)
 
-	// Wrap provisioner with extra information.
-	prov = wrapProvisioner(prov, attData)
+	// Wrap provisioner with extra information, if not nil
+	if prov != nil {
+		prov = wrapProvisioner(prov, attData)
+	}
 
 	// Store certificate in the db.
 	if err = a.storeCertificate(prov, cert); err != nil {