provider/aws: Fix issue in Classic env with external Security Groups

Linking a security group from another account requires setting the UserID, which we were stripping out
hashicorp · Jul 7, 2015 · cc79e6c · cc79e6c
1 parent b4c1c22
commit cc79e6c
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/builtin/providers/aws/structure.go b/builtin/providers/aws/structure.go
@@ -157,12 +157,15 @@ func expandIPPerms(
 
 				perm.UserIDGroupPairs[i] = &ec2.UserIDGroupPair{
 					GroupID: aws.String(id),
-					UserID:  aws.String(ownerId),
 				}
+
+				if ownerId != "" {
+					perm.UserIDGroupPairs[i].UserID = aws.String(ownerId)
+				}
+
 				if !vpc {
 					perm.UserIDGroupPairs[i].GroupID = nil
 					perm.UserIDGroupPairs[i].GroupName = aws.String(id)
-					perm.UserIDGroupPairs[i].UserID = nil
 				}
 			}
 		}