fix: Deprecated force_https

Author: gitmkn

tencentcloud/resource_tc_tse_cngw_route.go

@@ -67,7 +67,6 @@ resource "tencentcloud_tse_cngw_service" "cngw_service" {
 
 resource "tencentcloud_tse_cngw_route" "cngw_route" {
   destination_ports = []
-  force_https       = false
   gateway_id        = tencentcloud_tse_cngw_gateway.cngw_gateway.id
   hosts = [
     "192.168.0.1:9090",
@@ -191,6 +190,7 @@ func resourceTencentCloudTseCngwRoute() *schema.Resource {
 			"force_https": {
 				Optional:    true,
 				Type:        schema.TypeBool,
+				Deprecated:  "This field has been deprecated and will be deleted in subsequent versions.",
 				Description: "whether to enable forced HTTPS, no longer use.",
 			},
 
@@ -536,11 +536,11 @@ func resourceTencentCloudTseCngwRouteUpdate(d *schema.ResourceData, meta interfa
 		}
 	}
 
-	// if d.HasChange("force_https") {
-	// 	if v, ok := d.GetOkExists("force_https"); ok {
-	// 		request.ForceHttps = helper.Bool(v.(bool))
-	// 	}
-	// }
+	if d.HasChange("force_https") {
+		if v, ok := d.GetOkExists("force_https"); ok {
+			request.ForceHttps = helper.Bool(v.(bool))
+		}
+	}
 
 	if d.HasChange("destination_ports") {
 		if v, ok := d.GetOk("destination_ports"); ok {

tencentcloud/resource_tc_tse_cngw_route_test.go

@@ -26,7 +26,6 @@ func TestAccTencentCloudTseCngwRouteResource_basic(t *testing.T) {
 					testAccCheckTseCngwRouteExists("tencentcloud_tse_cngw_route.cngw_route"),
 					resource.TestCheckResourceAttrSet("tencentcloud_tse_cngw_route.cngw_route", "id"),
 					resource.TestCheckResourceAttr("tencentcloud_tse_cngw_route.cngw_route", "gateway_id", defaultTseGatewayId),
-					resource.TestCheckResourceAttr("tencentcloud_tse_cngw_route.cngw_route", "force_https", "false"),
 					resource.TestCheckResourceAttr("tencentcloud_tse_cngw_route.cngw_route", "hosts.#", "1"),
 					resource.TestCheckResourceAttr("tencentcloud_tse_cngw_route.cngw_route", "https_redirect_status_code", "426"),
 					resource.TestCheckResourceAttr("tencentcloud_tse_cngw_route.cngw_route", "paths.#", "1"),
@@ -51,7 +50,6 @@ func TestAccTencentCloudTseCngwRouteResource_basic(t *testing.T) {
 					testAccCheckTseCngwRouteExists("tencentcloud_tse_cngw_route.cngw_route"),
 					resource.TestCheckResourceAttrSet("tencentcloud_tse_cngw_route.cngw_route", "id"),
 					resource.TestCheckResourceAttr("tencentcloud_tse_cngw_route.cngw_route", "gateway_id", defaultTseGatewayId),
-					// resource.TestCheckResourceAttr("tencentcloud_tse_cngw_route.cngw_route", "force_https", "true"),
 					resource.TestCheckResourceAttr("tencentcloud_tse_cngw_route.cngw_route", "hosts.#", "1"),
 					resource.TestCheckResourceAttr("tencentcloud_tse_cngw_route.cngw_route", "https_redirect_status_code", "301"),
 					resource.TestCheckResourceAttr("tencentcloud_tse_cngw_route.cngw_route", "paths.#", "1"),
@@ -134,7 +132,6 @@ const testAccTseCngwRoute = DefaultTseVar + `
 
 resource "tencentcloud_tse_cngw_route" "cngw_route" {
   destination_ports = []
-  force_https       = false
   gateway_id        = var.gateway_id
   hosts = [
     "192.168.0.1:9090",
@@ -162,7 +159,6 @@ const testAccTseCngwRouteUp = DefaultTseVar + `
 
 resource "tencentcloud_tse_cngw_route" "cngw_route" {
   destination_ports = []
-  force_https       = false
   gateway_id        = var.gateway_id
   hosts = [
     "192.168.0.1:9091",

website/docs/r/tse_cngw_route.html.markdown

@@ -77,7 +77,6 @@ resource "tencentcloud_tse_cngw_service" "cngw_service" {
 
 resource "tencentcloud_tse_cngw_route" "cngw_route" {
   destination_ports = []
-  force_https       = false
   gateway_id        = tencentcloud_tse_cngw_gateway.cngw_gateway.id
   hosts = [
     "192.168.0.1:9090",
@@ -108,7 +107,7 @@ The following arguments are supported:
 * `gateway_id` - (Required, String) gateway ID.
 * `service_id` - (Required, String) ID of the service which the route belongs to.
 * `destination_ports` - (Optional, Set: [`Int`]) destination port for Layer 4 matching.
-* `force_https` - (Optional, Bool) whether to enable forced HTTPS, no longer use.
+* `force_https` - (Optional, Bool, **Deprecated**) This field has been deprecated and will be deleted in subsequent versions. whether to enable forced HTTPS, no longer use.
 * `headers` - (Optional, List) the headers of route.
 * `hosts` - (Optional, Set: [`String`]) host list.
 * `https_redirect_status_code` - (Optional, Int) https redirection status code.