From ac8320267a179d52e113390760d7f26653df7845 Mon Sep 17 00:00:00 2001
From: 205101-retr0 <ande.teja@learner.manipal.edu>
Date: Thu, 26 Oct 2023 10:15:32 +0530
Subject: [PATCH 1/3] added fix for terraform refresh

---
 fmc/resource_fmc_staticIPv4_route.go | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/fmc/resource_fmc_staticIPv4_route.go b/fmc/resource_fmc_staticIPv4_route.go
index 710963c..04ad86f 100644
--- a/fmc/resource_fmc_staticIPv4_route.go
+++ b/fmc/resource_fmc_staticIPv4_route.go
@@ -2,6 +2,7 @@ package fmc
 
 import (
 	"context"
+	"strings"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
@@ -241,15 +242,21 @@ func resourceFmcStaticIPv4RouteRead(ctx context.Context, d *schema.ResourceData,
 	// Warning or errors can be collected in a slice type
 	var diags diag.Diagnostics
 
+	// Why this GetStatic IPv4 route but others FmcStaticIPv4
 	item, err := c.GetFmcStaticIPv4Route(ctx, d.Get("device_id").(string), d.Id())
 	if err != nil {
-		diags = append(diags, diag.Diagnostic{
-			Severity: diag.Error,
-			Summary:  "unable to read route",
-			Detail:   err.Error(),
-		})
+		if strings.Contains(err.Error(), "404") {
+			d.SetId("")
+		} else {
+			diags = append(diags, diag.Diagnostic{
+				Severity: diag.Error,
+				Summary:  "unable to read route",
+				Detail:   err.Error(),
+			})
+		}
 		return diags
 	}
+
 	if err := d.Set("interface_name", item.InterfaceName); err != nil {
 		diags = append(diags, diag.Diagnostic{
 			Severity: diag.Error,

From 1fe8f87f255a1f36cd08eb656f2b623bb4daa0da Mon Sep 17 00:00:00 2001
From: Suraj <snetlamu@cisco.com>
Date: Mon, 6 Nov 2023 14:57:21 +0530
Subject: [PATCH 2/3] Resolves #131

---
 docs/resources/security_zone.md   |  1 +
 fmc/resource_fmc_security_zone.go | 15 +++++++++++++++
 2 files changed, 16 insertions(+)

diff --git a/docs/resources/security_zone.md b/docs/resources/security_zone.md
index 59ee38c..65f0130 100644
--- a/docs/resources/security_zone.md
+++ b/docs/resources/security_zone.md
@@ -39,5 +39,6 @@ resource "fmc_security_zone" "test" {
 ### Read-Only
 
 - `id` (String) The ID of this resource.
+- `type` (String) The type of this resource
 
 
diff --git a/fmc/resource_fmc_security_zone.go b/fmc/resource_fmc_security_zone.go
index dec4262..96f3571 100644
--- a/fmc/resource_fmc_security_zone.go
+++ b/fmc/resource_fmc_security_zone.go
@@ -28,6 +28,11 @@ func resourceFmcSecurityZone() *schema.Resource {
 		UpdateContext: resourceFmcSecurityZoneUpdate,
 		DeleteContext: resourceFmcSecurityZoneDelete,
 		Schema: map[string]*schema.Schema{
+			"type": {
+				Type:        schema.TypeString,
+				Computed:    true,
+				Description: "The type of this resource",
+			},
 			"name": {
 				Type:        schema.TypeString,
 				Required:    true,
@@ -100,6 +105,16 @@ func resourceFmcSecurityZoneRead(ctx context.Context, d *schema.ResourceData, m
 			return diags
 		}
 	}
+
+	if err := d.Set("type", item.Type); err != nil {
+		diags = append(diags, diag.Diagnostic{
+			Severity: diag.Error,
+			Summary:  "unable to read security zone",
+			Detail:   err.Error(),
+		})
+		return diags
+	}
+
 	if err := d.Set("name", item.Name); err != nil {
 		diags = append(diags, diag.Diagnostic{
 			Severity: diag.Error,

From 4f12168ac3362109fa7dce08233269851443b235 Mon Sep 17 00:00:00 2001
From: Suraj <snetlamu@cisco.com>
Date: Mon, 6 Nov 2023 17:08:56 +0530
Subject: [PATCH 3/3] Fixes #139

---
 .DS_Store                                     | Bin 6148 -> 6148 bytes
 examples/fmc_device_sub_interfaces/main.tf    |  50 +++++++++---------
 fmc/fmc_device_physicalinterfaces.go          |   6 +--
 fmc/resource_fmc_device_physicalinterfaces.go |   3 +-
 fmc/resource_fmc_device_subinterfaces.go      |  32 +++++------
 fmc/resource_fmc_device_vni.go                |   5 +-
 6 files changed, 50 insertions(+), 46 deletions(-)

diff --git a/.DS_Store b/.DS_Store
index edad4a235d77862a6c54328033a5ac836c5c82a6..6f28bd7eb3856f2f7f0ba6382b82d6db1d7b24e7 100644
GIT binary patch
delta 353
zcmZoMXfc=|#>B)qF;R?_gMop8V`8F+EEA9c0?a_n$iN`Lz)--D!;s2QoK#+1urYBt
zV?9WQ11M6#kjRjWESr>*p9GZKkyMbASzKaZaE+0PnT3^&or9f&n<F+jBfmVjB(bEl
z*eS6n8pI39&(BGMu@jTRGE>Xr1w@?l^GXsk^HPhz8ZuK-fl6Y+GxJjN%boH|^HS=I
z!J301G8~*7obdt@)z!vEraB6ymIk#t3e}cI2098R#%8s(oE)Oc`qn}5**Up+`CUNA
z0Rbb>TVTKorD0Szkb&W`vf!e;ocz3Wpd3gaGs8qfv5lE8SSA~Yux#cAdJGtV8xy}X
WPv#d<WCaDD!em1c>CFKmTbKdS$y8?m

delta 80
zcmZoMXfc=|#>AjHu~2w2Gqa2!69WV=0x2LCVAw3c{GDa90rL)~&Fmcf96+U;6`8*?
YPv#d<<OC^70I8pB!lS%7Mq~vu05-x7TmS$7

diff --git a/examples/fmc_device_sub_interfaces/main.tf b/examples/fmc_device_sub_interfaces/main.tf
index 2e88507..efb8d84 100644
--- a/examples/fmc_device_sub_interfaces/main.tf
+++ b/examples/fmc_device_sub_interfaces/main.tf
@@ -8,19 +8,19 @@ terraform {
 }
 
 provider "fmc" {
-  fmc_username = var.fmc_username
-  fmc_password = var.fmc_password
-  fmc_host = var.fmc_host
+  fmc_username             = var.fmc_username
+  fmc_password             = var.fmc_password
+  fmc_host                 = var.fmc_host
   fmc_insecure_skip_verify = var.fmc_insecure_skip_verify
 }
 
 data "fmc_devices" "device" {
-    name = "FTD1"
+  name = "FTD1"
 }
 
 resource "fmc_security_zone" "outside" {
-  name            = "outside"
-  interface_mode  = "ROUTED"
+  name           = "outside"
+  interface_mode = "ROUTED"
 }
 
 
@@ -30,28 +30,30 @@ resource "fmc_security_zone" "outside" {
 # }
 
 resource "fmc_device_subinterfaces" "sub" {
-    device_id   = data.fmc_devices.device.id
-    ifname = "Testing12"
-    subinterface_id = 12345
-    vlan_id = 80
-    name = "GigabitEthernet0/1"
-    mode = "NONE"
-    mtu = 1600
-    enabled = true
-    priority = 69
-    security_zone_id = fmc_security_zone.outside.id
-    ipv4_dhcp_enabled = false
-    ipv4_dhcp_route_metric = 1
-
-    enable_ipv6 = true
-    ipv6_address = "2001:10:240:ac::a"
-    ipv6_prefix = "124"
-    ipv6_enforce_eui = false
+  device_id              = data.fmc_devices.device.id
+  ifname                 = "Testing12"
+  subinterface_id        = 12345
+  vlan_id                = 80
+  name                   = "GigabitEthernet0/1"
+  mode                   = "NONE"
+  mtu                    = 1600
+  enabled                = true
+  priority               = 69
+  security_zone_id       = fmc_security_zone.outside.id
+  ipv4_dhcp_enabled      = false
+  ipv4_dhcp_route_metric = 1
+  ipv4_static_address    = "192.168.0.10"
+  ipv4_static_netmask    = 24
+
+  enable_ipv6      = true
+  ipv6_address     = "2001:10:240:ac::a"
+  ipv6_prefix      = "124"
+  ipv6_enforce_eui = false
 
 }
 
 output "new_subinterface_sub" {
-    value = fmc_device_subinterfaces.sub
+  value = fmc_device_subinterfaces.sub
 }
 
 # output "old_physical_interfaces" {
diff --git a/fmc/fmc_device_physicalinterfaces.go b/fmc/fmc_device_physicalinterfaces.go
index 6b60c06..e3feb40 100644
--- a/fmc/fmc_device_physicalinterfaces.go
+++ b/fmc/fmc_device_physicalinterfaces.go
@@ -17,8 +17,8 @@ type IPv6Address struct {
 }
 
 type IPv6 struct {
-	EnableIPv6    bool 	    `json:"enableIPV6,omitempty"`
-	Addresses []IPv6Address `json:"addresses,omitempty"`
+	EnableIPv6 bool          `json:"enableIPV6,omitempty"`
+	Addresses  []IPv6Address `json:"addresses,omitempty"`
 }
 
 // IPv4 Structs
@@ -30,7 +30,7 @@ type IPv4DHCP struct {
 
 type IPv4Static struct {
 	Address string `json:"address,omitempty"`
-	Netmask int    `json:"netmask,omitempty"`
+	Netmask string `json:"netmask,omitempty"`
 }
 
 type IPv4 struct {
diff --git a/fmc/resource_fmc_device_physicalinterfaces.go b/fmc/resource_fmc_device_physicalinterfaces.go
index 83c1f1f..1b9849e 100644
--- a/fmc/resource_fmc_device_physicalinterfaces.go
+++ b/fmc/resource_fmc_device_physicalinterfaces.go
@@ -3,6 +3,7 @@ package fmc
 import (
 	"context"
 	"log"
+	"strconv"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
@@ -207,7 +208,7 @@ func resourcePhyInterfaceUpdate(ctx context.Context, d *schema.ResourceData, m i
 
 	var IPv4Static = IPv4Static{
 		Address: ipv4StaticAddress,
-		Netmask: ipv4StaticNetmask,
+		Netmask: strconv.Itoa(ipv4StaticNetmask),
 	}
 	var IPv4DHCP = IPv4DHCP{
 		Enable:      ipv4DhcpEnabled,
diff --git a/fmc/resource_fmc_device_subinterfaces.go b/fmc/resource_fmc_device_subinterfaces.go
index 3e40ae2..0eca812 100644
--- a/fmc/resource_fmc_device_subinterfaces.go
+++ b/fmc/resource_fmc_device_subinterfaces.go
@@ -2,6 +2,7 @@ package fmc
 
 import (
 	"context"
+	"strconv"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
@@ -73,10 +74,10 @@ func resourceFmcSubInterface() *schema.Resource {
 				Computed:    true,
 				Description: "The type of this resource",
 			},
-			"mode": { 
+			"mode": {
 				Type:        schema.TypeString,
 				Optional:    true,
-				Default: 	 "NONE",
+				Default:     "NONE",
 				Description: "The mode of this resource",
 			},
 			"name": {
@@ -93,7 +94,7 @@ func resourceFmcSubInterface() *schema.Resource {
 			"priority": {
 				Type:        schema.TypeInt,
 				Optional:    true,
-				Default: 	 0,
+				Default:     0,
 				Description: "The type of this resource",
 			},
 			"security_zone_id": {
@@ -193,14 +194,13 @@ func resourceFmcSubInterfaceCreate(ctx context.Context, d *schema.ResourceData,
 
 	var diags diag.Diagnostics
 
-
 	ipv4StaticAddress := d.Get("ipv4_static_address").(string)
 	ipv4StaticNetmask := d.Get("ipv4_static_netmask").(int)
 	ipv4DhcpEnabled := d.Get("ipv4_dhcp_enabled").(bool)
 	ipv4DhcpRouteMetric := d.Get("ipv4_dhcp_route_metric").(int)
-	
+
 	securityZoneId := d.Get("security_zone_id").(string)
-	
+
 	ipv6Address := d.Get("ipv6_address").(string)
 	ipv6Prefix := d.Get("ipv6_prefix").(int)
 	ipv6EnforceEUI := d.Get("ipv6_enforce_eui").(bool)
@@ -215,7 +215,7 @@ func resourceFmcSubInterfaceCreate(ctx context.Context, d *schema.ResourceData,
 		})
 	}
 
-	var IPv6 = IPv6{EnableIPv6:enable_ipv6 ,Addresses: IPv6Add}
+	var IPv6 = IPv6{EnableIPv6: enable_ipv6, Addresses: IPv6Add}
 
 	var SubInterfaceSecurityZone = PhysicalInterfaceSecurityZone{
 		ID:   securityZoneId,
@@ -223,7 +223,7 @@ func resourceFmcSubInterfaceCreate(ctx context.Context, d *schema.ResourceData,
 	}
 	var IPv4Static = IPv4Static{
 		Address: ipv4StaticAddress,
-		Netmask: ipv4StaticNetmask,
+		Netmask: strconv.Itoa(ipv4StaticNetmask),
 	}
 	var IPv4DHCP = IPv4DHCP{
 		Enable:      ipv4DhcpEnabled,
@@ -242,14 +242,14 @@ func resourceFmcSubInterfaceCreate(ctx context.Context, d *schema.ResourceData,
 		Mode:           d.Get("mode").(string),
 		Name:           d.Get("name").(string),
 		IPv4:           IPv4,
-		VlanID: 	    d.Get("vlan_id").(int),
+		VlanID:         d.Get("vlan_id").(int),
 		SubInterfaceID: d.Get("subinterface_id").(int),
 		Enabled:        d.Get("enabled").(bool),
 		SecurityZone:   SubInterfaceSecurityZone,
 		MgmntOnly:      d.Get("management_only").(bool),
 		Priority:       d.Get("priority").(int),
 		MTU:            d.Get("mtu").(int),
-		IPv6: IPv6,
+		IPv6:           IPv6,
 	})
 	if err != nil {
 		diags = append(diags, diag.Diagnostic{
@@ -268,7 +268,7 @@ func resourceFmcSubInterfaceUpdate(ctx context.Context, d *schema.ResourceData,
 
 	var diags diag.Diagnostics
 	id := d.Id()
-	if d.HasChanges("ipv6_enforce_eui","ipv6_prefix","ipv6_address","vlan_id","management_only","ifname","name", "mode", "ipv4_static_address", "security_zone_id", "ipv4_dhcp_enabled","ipv4_dhcp_route_metric", "priority", "enabled") {
+	if d.HasChanges("ipv6_enforce_eui", "ipv6_prefix", "ipv6_address", "vlan_id", "management_only", "ifname", "name", "mode", "ipv4_static_address", "ipv4_static_netmask", "security_zone_id", "ipv4_dhcp_enabled", "ipv4_dhcp_route_metric", "priority", "enabled") {
 
 		ipv4StaticAddress := d.Get("ipv4_static_address").(string)
 		ipv4StaticNetmask := d.Get("ipv4_static_netmask").(int)
@@ -283,7 +283,7 @@ func resourceFmcSubInterfaceUpdate(ctx context.Context, d *schema.ResourceData,
 		}
 		var IPv4Static = IPv4Static{
 			Address: ipv4StaticAddress,
-			Netmask: ipv4StaticNetmask,
+			Netmask: strconv.Itoa(ipv4StaticNetmask),
 		}
 		var IPv4DHCP = IPv4DHCP{
 			Enable:      ipv4DhcpEnabled,
@@ -311,7 +311,7 @@ func resourceFmcSubInterfaceUpdate(ctx context.Context, d *schema.ResourceData,
 			})
 		}
 
-		var IPv6 = IPv6{EnableIPv6:enable_ipv6 ,Addresses: IPv6Add}
+		var IPv6 = IPv6{EnableIPv6: enable_ipv6, Addresses: IPv6Add}
 
 		_, err := c.UpdateFmcSubInterface(ctx, d.Get("device_id").(string), id, &SubInterface{
 			ID:             id,
@@ -319,14 +319,14 @@ func resourceFmcSubInterfaceUpdate(ctx context.Context, d *schema.ResourceData,
 			Mode:           d.Get("mode").(string),
 			Name:           d.Get("name").(string),
 			IPv4:           IPv4,
-			VlanID: 	    d.Get("vlan_id").(int),
+			VlanID:         d.Get("vlan_id").(int),
 			Enabled:        d.Get("enabled").(bool),
 			SecurityZone:   SubInterfaceSecurityZone,
 			SubInterfaceID: d.Get("subinterface_id").(int),
 			MgmntOnly:      d.Get("management_only").(bool),
 			Priority:       d.Get("priority").(int),
 			MTU:            d.Get("mtu").(int),
-			IPv6:          IPv6,
+			IPv6:           IPv6,
 		})
 		if err != nil {
 			diags = append(diags, diag.Diagnostic{
@@ -348,7 +348,7 @@ func resourceFmcSubInterfaceDelete(ctx context.Context, d *schema.ResourceData,
 
 	id := d.Id()
 
-	err := c.DeleteFmcSubInterface(ctx, d.Get("device_id").(string) ,id)
+	err := c.DeleteFmcSubInterface(ctx, d.Get("device_id").(string), id)
 	if err != nil {
 		diags = append(diags, diag.Diagnostic{
 			Severity: diag.Error,
diff --git a/fmc/resource_fmc_device_vni.go b/fmc/resource_fmc_device_vni.go
index c0f3235..afdd8de 100644
--- a/fmc/resource_fmc_device_vni.go
+++ b/fmc/resource_fmc_device_vni.go
@@ -3,6 +3,7 @@ package fmc
 import (
 	"context"
 	"log"
+	"strconv"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
@@ -223,7 +224,7 @@ func resourceVNICreate(ctx context.Context, d *schema.ResourceData, m interface{
 					static := statics[0].(map[string]interface{})
 					var IPv4Static = IPv4Static{
 						Address: static["address"].(string),
-						Netmask: static["netmask"].(int),
+						Netmask: strconv.Itoa(static["netmask"].(int)),
 					}
 					ipv4.Static = &IPv4Static
 					isStatic = true
@@ -324,7 +325,7 @@ func resourceVNIUpdate(ctx context.Context, d *schema.ResourceData, m interface{
 						static := statics[0].(map[string]interface{})
 						var IPv4Static = IPv4Static{
 							Address: static["address"].(string),
-							Netmask: static["netmask"].(int),
+							Netmask: strconv.Itoa(static["netmask"].(int)),
 						}
 						log.Printf("IPv4Static=%v", IPv4Static)
 						ipv4.Static = &IPv4Static