Merge pull request #34737 from thulasirajkomminar/b-aws_glue_connecti…

…on-empty-physical_connection_requirements Fixing empty `physical_connection_requirements` in `aws_glue_connection` resource
hashicorp · Dec 5, 2023 · 8a5d84d · 8a5d84d
2 parents fccb41c + b51dec1
commit 8a5d84d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/.changelog/34737.txt b/.changelog/34737.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+resource/aws_glue_connection: Fix crash while creating resource with empty `physical_connection_requirements` configuration block
+```
diff --git a/internal/service/glue/connection.go b/internal/service/glue/connection.go
@@ -277,9 +277,8 @@ func expandConnectionInput(d *schema.ResourceData) *glue.ConnectionInput {
 		connectionInput.MatchCriteria = flex.ExpandStringList(v.([]interface{}))
 	}
 
-	if v, ok := d.GetOk("physical_connection_requirements"); ok {
-		physicalConnectionRequirementsList := v.([]interface{})
-		physicalConnectionRequirementsMap := physicalConnectionRequirementsList[0].(map[string]interface{})
+	if v, ok := d.GetOk("physical_connection_requirements"); ok && v.([]interface{})[0] != nil {
+		physicalConnectionRequirementsMap := v.([]interface{})[0].(map[string]interface{})
 		connectionInput.PhysicalConnectionRequirements = expandPhysicalConnectionRequirements(physicalConnectionRequirementsMap)
 	}