correctly append to region/namespace slices while handling nil interfaces

Author: apoorv-garg

sumologic/resource_sumologic_generic_polling_source.go

@@ -335,18 +335,18 @@ func getPollingPathSettings(d *schema.ResourceData) (PollingPath, error) {
 		case "CloudWatchPath", "AwsInventoryPath":
 			pathSettings.Type = pathType
 			rawLimitToRegions := path["limit_to_regions"].([]interface{})
-			LimitToRegions := make([]string, len(rawLimitToRegions))
-			for i, v := range rawLimitToRegions {
+			LimitToRegions := make([]string, 0, len(rawLimitToRegions))
+			for _, v := range rawLimitToRegions {
 				if v != nil {
-					LimitToRegions[i] = v.(string)
+					LimitToRegions = append(LimitToRegions, v.(string))
 				}
 			}
 
 			rawLimitToNamespaces := path["limit_to_namespaces"].([]interface{})
-			LimitToNamespaces := make([]string, len(rawLimitToNamespaces))
-			for i, v := range rawLimitToNamespaces {
+			LimitToNamespaces := make([]string, 0, len(rawLimitToNamespaces))
+			for _, v := range rawLimitToNamespaces {
 				if v != nil {
-					LimitToNamespaces[i] = v.(string)
+					LimitToNamespaces = append(LimitToNamespaces, v.(string))
 				}
 			}
 			pathSettings.LimitToRegions = LimitToRegions
@@ -357,10 +357,10 @@ func getPollingPathSettings(d *schema.ResourceData) (PollingPath, error) {
 		case "AwsXRayPath":
 			pathSettings.Type = "AwsXRayPath"
 			rawLimitToRegions := path["limit_to_regions"].([]interface{})
-			LimitToRegions := make([]string, len(rawLimitToRegions))
-			for i, v := range rawLimitToRegions {
+			LimitToRegions := make([]string, 0, len(rawLimitToRegions))
+			for _, v := range rawLimitToRegions {
 				if v != nil {
-					LimitToRegions[i] = v.(string)
+					LimitToRegions = append(LimitToRegions, v.(string))
 				}
 			}
 			pathSettings.LimitToRegions = LimitToRegions