Merge pull request #168 from Sherlock-Holo/gaap-fix-realserver-auth

fix sometimes can't enable realserver auth

Author: likexian

CHANGELOG.md

@@ -5,6 +5,10 @@ ENHANCEMENTS:
 * Resource: `tencentcloud_gaap_http_domain` support import
 * Resource: `tencentcloud_gaap_layer7_listener` support import
 
+BUG FIXES:
+
+* Resource: `tencentcloud_gaap_http_domain` fix sometimes can't enable realserver auth
+
 ## 1.20.1 (October 08, 2019)
 
 ENHANCEMENTS:

tencentcloud/resource_tc_gaap_http_domain.go

@@ -419,6 +419,20 @@ func resourceTencentCloudGaapHttpDomainUpdate(d *schema.ResourceData, m interfac
 	if d.HasChange("realserver_auth") {
 		updateAdvancedAttr = append(updateAdvancedAttr, "realserver_auth")
 		realserverAuth = boolToPointer(d.Get("realserver_auth").(bool))
+
+		if *realserverAuth {
+			if _, ok := d.GetOk("realserver_certificate_id"); !ok {
+				return errors.New("when enable realserver auth, realserver_certificate_id must be set")
+			}
+
+			if _, ok := d.GetOk("realserver_certificate_domain"); !ok {
+				return errors.New("when enable realserver auth, realserver_certificate_domain must be set")
+			}
+
+			// if enable realserver auth, must send realserverCertificateId and realserverCertificateDomain
+			realserverCertificateId = stringToPointer(d.Get("realserver_certificate_id").(string))
+			realserverCertificateDomain = stringToPointer(d.Get("realserver_certificate_domain").(string))
+		}
 	}
 	if d.HasChange("realserver_certificate_id") {
 		updateAdvancedAttr = append(updateAdvancedAttr, "realserver_certificate_id")