From df608e8928e88f5d8a16b21c84a39250abeefe6a Mon Sep 17 00:00:00 2001 From: Sam Levenick Date: Wed, 17 Jul 2019 17:47:18 +0000 Subject: [PATCH] Add check on import name split length Signed-off-by: Modular Magician --- google/resource_access_context_manager_access_level.go | 3 +++ google/resource_access_context_manager_service_perimeter.go | 3 +++ 2 files changed, 6 insertions(+) diff --git a/google/resource_access_context_manager_access_level.go b/google/resource_access_context_manager_access_level.go index 83430a6fc47..7e24d01e5c6 100644 --- a/google/resource_access_context_manager_access_level.go +++ b/google/resource_access_context_manager_access_level.go @@ -365,6 +365,9 @@ func resourceAccessContextManagerAccessLevelImport(d *schema.ResourceData, meta return nil, err } stringParts := strings.Split(d.Get("name").(string), "/") + if len(stringParts) < 2 { + return nil, fmt.Errorf("Error parsing parent name. Should be in form accessPolicies/{{policy_id}}/access_level/{{short_name}}") + } d.Set("parent", fmt.Sprintf("%s/%s", stringParts[0], stringParts[1])) return []*schema.ResourceData{d}, nil } diff --git a/google/resource_access_context_manager_service_perimeter.go b/google/resource_access_context_manager_service_perimeter.go index 73b91f13e25..dcc899423ec 100644 --- a/google/resource_access_context_manager_service_perimeter.go +++ b/google/resource_access_context_manager_service_perimeter.go @@ -332,6 +332,9 @@ func resourceAccessContextManagerServicePerimeterImport(d *schema.ResourceData, return nil, err } stringParts := strings.Split(d.Get("name").(string), "/") + if len(stringParts) < 2 { + return nil, fmt.Errorf("Error parsing parent name. Should be in form accessPolicies/{{policy_id}}/service_perimeter/{{short_name}}") + } d.Set("parent", fmt.Sprintf("%s/%s", stringParts[0], stringParts[1])) return []*schema.ResourceData{d}, nil }