From 4cf299b0fdf94684c2485b51d2d726f2cc7608d8 Mon Sep 17 00:00:00 2001
From: wanyaoqi <wanyaoqi@yunionyun.com>
Date: Wed, 22 Apr 2020 21:59:43 +0800
Subject: [PATCH] fix check ca cert

---
 pkg/keystone/models/servicecertificates.go | 30 ++++++++++++++++------
 1 file changed, 22 insertions(+), 8 deletions(-)

diff --git a/pkg/keystone/models/servicecertificates.go b/pkg/keystone/models/servicecertificates.go
index cebd60c433..732931e5fc 100644
--- a/pkg/keystone/models/servicecertificates.go
+++ b/pkg/keystone/models/servicecertificates.go
@@ -47,10 +47,17 @@ func (man *SServiceCertificateManager) ValidateCreateData(ctx context.Context, u
 	}
 	data = v.UpdateCertKeyInfo(ctx, data)
 
-	// validate ca cert key
-	v = validators.NewCertKeyValidator("ca_certificate", "ca_private_key")
-	if err := v.Validate(data); err != nil {
-		return nil, err
+	if caCert, _ := data.GetString("ca_certificate"); len(caCert) > 0 {
+		vc := validators.NewCertificateValidator("ca_certificate")
+		if err := vc.Validate(data); err != nil {
+			return nil, err
+		}
+	}
+	if caPkey, _ := data.GetString("ca_private_key"); len(caPkey) > 0 {
+		vp := validators.NewPrivateKeyValidator("ca_private_key")
+		if err := vp.Validate(data); err != nil {
+			return nil, err
+		}
 	}
 
 	input := apis.StandaloneResourceCreateInput{}
@@ -76,10 +83,17 @@ func (cert *SServiceCertificate) ValidateUpdateData(
 	}
 	data = v.UpdateCertKeyInfo(ctx, data)
 
-	// validate ca cert key
-	v = validators.NewCertKeyValidator("ca_certificate", "ca_private_key")
-	if err := v.Validate(data); err != nil {
-		return nil, err
+	if caCert, _ := data.GetString("ca_certificate"); len(caCert) > 0 {
+		vc := validators.NewCertificateValidator("ca_certificate")
+		if err := vc.Validate(data); err != nil {
+			return nil, err
+		}
+	}
+	if caPkey, _ := data.GetString("ca_private_key"); len(caPkey) > 0 {
+		vp := validators.NewPrivateKeyValidator("ca_private_key")
+		if err := vp.Validate(data); err != nil {
+			return nil, err
+		}
 	}
 
 	updateData := jsonutils.NewDict()